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
Upgrade to Lucene 9.9.0 #102782
Upgrade to Lucene 9.9.0 #102782
Conversation
Adds new max_inner_product vector similarity function. This differs from dot_product in the following ways: Doesn't require vectors to be normalized Scales the similarity between vectors differently to prevent negative scores
This adds a version for the Lucene upgrade and adjusts some tests.
…#99962) Fix compiling error after adding a new method to RandomAccessInput interface.
Pinging @elastic/es-search (Team:Search) |
Hi @ChrisHegarty, I've created a changelog YAML for you. |
server/src/main/java/org/elasticsearch/index/IndexVersions.java
Outdated
Show resolved
Hide resolved
@elasticsearchmachine run elasticsearch-ci/part-1 |
Quantized search result scores can be slightly different depending on if they are merged into a single segment, etc.
68fd12c
to
eaa5889
Compare
This all looks good to me, besides one small thing: I think we are missing a changelog for the " Automatically quantize vectors for kNN search" change, is that possible @benwtrent ? |
@javanna you are correct, let me commit it really quick |
@elasticsearchmachine run elasticsearch-ci/part-1 |
Matcher matcher; | ||
if ("UPGRADE_TO_LUCENE_9_9".equals(field.getName())) { | ||
// OK | ||
} else if ((matcher = historicalVersion.matcher(field.getName())).matches()) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the only thing I don't really understand. But the PR is ++ from me!
Upgrade to Lucene 9.9.0
TODO:
Note: This PR will merge it's commits into main (rather than squashing), in order to preserve history and already-reviewed changes.