Change imports to align with core lucene upgrade#1147
Change imports to align with core lucene upgrade#1147navneet1v merged 1 commit intoopensearch-project:mainfrom
Conversation
Signed-off-by: Ryan Bogan <rbogan@amazon.com>
|
This has not been backported to 2.x on core, so there is no need for us to backport either. |
|
Can we skip the CHANGELOG on this? |
yes I just added skip-changelog label |
Shouldn't we put it as Maintenance? |
Codecov Report
@@ 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
|
|
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. |
@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 |
Signed-off-by: Ryan Bogan <rbogan@amazon.com> (cherry picked from commit ca5e483)
…1147) Signed-off-by: Ryan Bogan <rbogan@amazon.com>
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
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.