Skip to content

Conversation

@ioanatia
Copy link
Contributor

@ioanatia ioanatia commented Jul 2, 2025

related #123389
We know we will not have a RRF command, but rather another command called FUSE that will handle both reciprocal rank fusion and linear combination.

We have a very high degree of confidence that FUSE with no options will apply reciprocal rank fusion by default.
We also know that unlike the RRF command, FUSE will not order the results.

I am adding FUSE as pretty much a copy of RRF. It's a copy for now and not a replacement.
Kibana already has autocomplete for the RRF command.
I checked with @stratoula that removing RRF in ES will cause failures in Kibana once we try to update the grammar files there.
Which is why for a brief moment we will have both FUSE and RRF.
Once we update the grammar files in Kibana, we can replace the autocomplete that we had for RRF to just be an initial version of the autocomplete for FUSE. After that we can just remove the RRF command in ES.

It was either going on this route of making FUSE a copy of RRF or reverting the autocomplete we have in Kibana for RRF.
I think this way is less intrusive.

@ioanatia ioanatia added >non-issue ES|QL-ui Impacts ES|QL UI :Search Relevance/Search Catch all for Search Relevance Team:Search - Relevance The Search organization Search Relevance team v9.2.0 labels Jul 2, 2025
@elasticsearchmachine elasticsearchmachine added the Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch label Jul 2, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/kibana-esql (ES|QL-ui)

@elasticsearchmachine elasticsearchmachine removed the Team:Search - Relevance The Search organization Search Relevance team label Jul 2, 2025
@@ -0,0 +1,136 @@
//
// CSV spec for FUSE command
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this is pretty much a copy of the RRF csv tests

Copy link
Contributor

@ChrisHegarty ChrisHegarty left a comment

Choose a reason for hiding this comment

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

LGTM

@ioanatia ioanatia merged commit fc59cd7 into elastic:main Jul 2, 2025
34 checks passed
@ioanatia ioanatia deleted the rrf_to_fuse branch July 2, 2025 17:39
mridula-s109 pushed a commit to mridula-s109/elasticsearch that referenced this pull request Jul 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ES|QL-ui Impacts ES|QL UI >non-issue :Search Relevance/Search Catch all for Search Relevance Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants