Skip to content

Add Lucene 9.12.1 index version constant #118557

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

Merged
merged 2 commits into from
Dec 14, 2024

Conversation

ChrisHegarty
Copy link
Contributor

This commit adds the IndexVersion constant for Lucene 9.12.1. Since main is already on Lucene 10, this change trivially just ensures that the constant version value in main is the same as that in the 8.x branch.

relates #118300

@ChrisHegarty ChrisHegarty added >non-issue Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch :Search Relevance/Search Catch all for Search Relevance labels Dec 12, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-search-relevance (Team:Search Relevance)

@@ -133,6 +133,7 @@ private static Version parseUnchecked(String version) {
public static final IndexVersion TIME_BASED_K_ORDERED_DOC_ID_BACKPORT = def(8_520_00_0, Version.LUCENE_9_12_0);
public static final IndexVersion V8_DEPRECATE_SOURCE_MODE_MAPPER = def(8_521_00_0, Version.LUCENE_9_12_0);
public static final IndexVersion USE_SYNTHETIC_SOURCE_FOR_RECOVERY_BACKPORT = def(8_522_00_0, Version.LUCENE_9_12_0);
public static final IndexVersion UPGRADE_TO_LUCENE_9_12_1 = def(8_523_00_0, parseUnchecked("9.12.1"));
Copy link
Contributor Author

@ChrisHegarty ChrisHegarty Dec 12, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We use parseUnchecked here as a temporary measure. Lucene 10.0 does not have the 9.12.1 version constant, the upcoming Lucene 10.1 will. (see apache/lucene#14060)

@ChrisHegarty ChrisHegarty requested a review from javanna December 13, 2024 11:37
@ChrisHegarty ChrisHegarty merged commit ce2a7de into elastic:main Dec 14, 2024
16 checks passed
@ChrisHegarty ChrisHegarty deleted the 9_12_1_index_constant branch December 14, 2024 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>non-issue :Search Relevance/Search Catch all for Search Relevance Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants