Skip to content

Conversation

@carlosdelest
Copy link
Member

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

(cherry picked from commit 5996772)

# Conflicts:
#	server/src/main/java/org/elasticsearch/search/vectors/KnnSearchBuilder.java
#	x-pack/plugin/rank-rrf/src/main/java/org/elasticsearch/xpack/rank/rrf/RRFRankBuilder.java
@carlosdelest carlosdelest added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Dec 11, 2024
@elasticsearchmachine elasticsearchmachine merged commit d40813d into elastic:8.x Dec 11, 2024
15 checks passed
@carlosdelest carlosdelest deleted the backport/8.x/pr-116663 branch December 11, 2024 10:17
@Override
public double doubleValue() throws IOException {
vectorOpsCount++;
return vectorSimilarityFunction.compare(target, vectorValues.vectorValue());
Copy link
Member Author

Choose a reason for hiding this comment

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

@benwtrent , FYI - this is different than the original PR as Lucene 10 has a different interface for FloatVectorValues vector retrieval.

I believe we're covered by testing and the checks I've done, but flagging for additional 👀 on it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport v8.18.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants