Update for Lucene 10.4

[mulugetam]

Description

Update for Lucene 10.4.

Related Issues

Resolves #312

Check List

• New functionality includes testing.
• Commits are signed per the DCO using --signoff.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

[mulugetam]

@reta @andrross Current main doesn't build. Please review the update for Lucene 10.4.

[mulugetam]

Integration tests fail b/c the test cluster distribution still has 10.3.2. Should work once OpenSearch-3.6.0-SNAPSHOT picks the Lucene 10.4.0 bump.

[ShawnQiang1]

@mulugetam I submitted a review comment, please take a look.

[andrross]

Thanks @mulugetam!

@reta Similar to my questions about the 2.19 bwc tests, what should the versions on main be? Currently it is this:

String previousVersion = System.getProperty("bwc.version.previous", "2.11.1.0")
String nextVersion = System.getProperty("bwc.version.next", "3.0.0.0")

which suggests we might not be testing the right thing here?

[reta]

Thanks @mulugetam , just a few nits please, LGTM otherwise

[reta]

Thanks @mulugetam!

@reta Similar to my questions about the 2.19 bwc tests, what should the versions on main be? Currently it is this:

String previousVersion = System.getProperty("bwc.version.previous", "2.11.1.0")
String nextVersion = System.getProperty("bwc.version.next", "3.0.0.0")

which suggests we might not be testing the right thing here?

Tnanks @andrross , we did a major overhaul in 2.12.x with codec naming, so we kept 2.11.x to make sure there are/were no regressions.

[andrross]

Tnanks @andrross , we did a major overhaul in 2.12.x with codec naming, so we kept 2.11.x to make sure there are/were no regressions.

@reta Actually, I think these defaults are not used because the CI workflow sets the branches which is then used to get the versions from those branches.

[mulugetam]

@reta I have applied the changes. Thanks!

[reta]

Tnanks @andrross , we did a major overhaul in 2.12.x with codec naming, so we kept 2.11.x to make sure there are/were no regressions.

@reta Actually, I think these defaults are not used because the CI workflow sets the branches which is then used to get the versions from those branches.

Ah, rIght, thanks @andrross