Skip to content

Change imports to align with core lucene upgrade#1147

Merged
navneet1v merged 1 commit intoopensearch-project:mainfrom
ryanbogan:lucene_changes
Sep 25, 2023
Merged

Change imports to align with core lucene upgrade#1147
navneet1v merged 1 commit intoopensearch-project:mainfrom
ryanbogan:lucene_changes

Conversation

@ryanbogan
Copy link
Copy Markdown
Member

Description

OpenSearch core recently updated the lucene version, rendering some constants used in k-NN obsolete. This PR updates to the new versions of those constants.

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed as 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.

Signed-off-by: Ryan Bogan <rbogan@amazon.com>
@ryanbogan
Copy link
Copy Markdown
Member Author

This has not been backported to 2.x on core, so there is no need for us to backport either.

@ryanbogan
Copy link
Copy Markdown
Member Author

Can we skip the CHANGELOG on this?

@navneet1v
Copy link
Copy Markdown
Collaborator

Can we skip the CHANGELOG on this?

yes I just added skip-changelog label

@heemin32
Copy link
Copy Markdown
Collaborator

Can we skip the CHANGELOG on this?

Shouldn't we put it as Maintenance?

@codecov
Copy link
Copy Markdown

codecov bot commented Sep 20, 2023

Codecov Report

Merging #1147 (6a82705) into main (ed7171e) will decrease coverage by 0.07%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main    #1147      +/-   ##
============================================
- Coverage     85.01%   84.95%   -0.07%     
  Complexity     1184     1184              
============================================
  Files           159      159              
  Lines          4813     4813              
  Branches        433      433              
============================================
- Hits           4092     4089       -3     
- Misses          525      528       +3     
  Partials        196      196              
Files Changed Coverage Δ
...opensearch/knn/index/mapper/LuceneFieldMapper.java 71.15% <ø> (ø)
.../java/org/opensearch/knn/index/util/KNNEngine.java 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

@martin-gaievski
Copy link
Copy Markdown
Member

did you check why bwc for rolling upgrade are failing? Also I wonder if we need to update our codec with this lucene version bump, I think current version of lucene codec is 9.5, and core has switched to 9.8

@ryanbogan
Copy link
Copy Markdown
Member Author

ryanbogan commented Sep 20, 2023

did you check why bwc for rolling upgrade are failing? Also I wonder if we need to update our codec with this lucene version bump, I think current version of lucene codec is 9.5, and core has switched to 9.8

It's failing with a connect exception, but that's not unique to this PR. I'm not sure what's causing that exception, but it's present on every PR I've raised recently.

As for the codec version, I'm not familiar with how frequently we update them. The core upgrade was from one snapshot of 9.8 to a newer snapshot of 9.8.

@navneet1v navneet1v merged commit ca5e483 into opensearch-project:main Sep 25, 2023
@navneet1v
Copy link
Copy Markdown
Collaborator

did you check why bwc for rolling upgrade are failing? Also I wonder if we need to update our codec with this lucene version bump, I think current version of lucene codec is 9.5, and core has switched to 9.8

@martin-gaievski we should move to latest codec. @heemin32 can we migrate to codec as per latest update of core, as you are the release owner for 2.11

@ryanbogan ryanbogan deleted the lucene_changes branch September 25, 2023 16:41
opensearch-trigger-bot bot pushed a commit that referenced this pull request Nov 6, 2023
Signed-off-by: Ryan Bogan <rbogan@amazon.com>
(cherry picked from commit ca5e483)
jingqimao77-spec pushed a commit to jingqimao77-spec/k-NN that referenced this pull request Mar 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants