Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Default CI Java Version to Java 11, run tests on 8, 14 (and 17) #288

Closed
3 tasks
Tracked by #64
dblock opened this issue Nov 17, 2021 · 1 comment · Fixed by #451
Closed
3 tasks
Tracked by #64

Default CI Java Version to Java 11, run tests on 8, 14 (and 17) #288

dblock opened this issue Nov 17, 2021 · 1 comment · Fixed by #451
Assignees
@joshuali925
Labels
v1.3.0

Comments

@dblock
Copy link
Member

dblock commented Nov 17, 2021

Coming from opensearch-project/opensearch-plugins#64, default CI Java version to 11 (LTS), and run tests on 8, 14 and 17.

  • Change the default Java version for CI in GitHub actions from 14/15 to 11 (LTS).
  • Add Java 8, 14 (used before), and 17 (next LTS) to the CI matrix.
  • Document Java versions in DEVELOPER_GUIDE.md.
@dblock dblock mentioned this issue Nov 17, 2021
Closed
15 tasks
@davidcui1225 davidcui1225 mentioned this issue Mar 1, 2022
Merged
6 tasks
@dblock
Copy link
Member Author

dblock commented Mar 9, 2022

@davidcui1225 You still have to restore building on JDK8 for 1.3.0.

@dblock dblock reopened this Mar 9, 2022
@dblock dblock changed the title Default CI Java Version to Java 11, run tests on 8, 14 and 17 Default CI Java Version to Java 11, run tests on 8, 14 (and 17) Mar 10, 2022
@dblock dblock removed the untriaged label Mar 10, 2022
@acarbonetto acarbonetto mentioned this issue Feb 17, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@joshuali925 joshuali925

Labels
v1.3.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add CI Matrix for JDK 11 and 14 davidcui1225/sql-1
3 participants
@dblock @joshuali925 @davidcui1225