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.

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

[FEATURE] Set opensearch-hadoop 2.0.0 baseline JDK version to JDK-21 #472

Open
Tracked by #14011
reta opened this issue Jun 5, 2024 · 4 comments
Open
Tracked by #14011

[FEATURE] Set opensearch-hadoop 2.0.0 baseline JDK version to JDK-21 #472

reta opened this issue Jun 5, 2024 · 4 comments
Labels
enhancement New feature or request

Comments

@reta
Copy link
Contributor

reta commented Jun 5, 2024

Is your feature request related to a problem? Please describe

Set opensearch-hadoop 2.0.0 baseline JDK version to JDK-21

Describe the solution you'd like

See please opensearch-project/OpenSearch#10745

Related component

Build

Describe alternatives you've considered

N/A

Additional context

See please opensearch-project/OpenSearch#14011

@Xtansia
Copy link
Collaborator

Xtansia commented Jun 5, 2024

@reta Are you sure this applies here? As this is a client shouldn't it be in the same boat as the Java client and trying to preserve as much backwards compatibility as possible?

@reta
Copy link
Contributor Author

reta commented Jun 5, 2024

@reta Are you sure this applies here? As this is a client shouldn't it be in the same boat as the Java client and trying to preserve as much backwards compatibility as possible?

Thanks @Xtansia , I am pretty sure it does apply (sadly) unless there is no goal to support Apache Lucene 10 by opensearch-hadoop [1], the baseline comes from there.

[1] https://github.com/opensearch-project/opensearch-hadoop/blob/main/buildSrc/opensearch-hadoop-version.properties#L3

@Xtansia
Copy link
Collaborator

Xtansia commented Jun 10, 2024

@reta I believe those might only used for integration tests, but would need to confirm

@dblock dblock removed the untriaged label Jul 1, 2024
@dblock
Copy link
Member

dblock commented Jul 1, 2024

[Catch All Triage - Attendees 1, 2, 3, 4, 5]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants