-
Notifications
You must be signed in to change notification settings - Fork 25.5k
ES|QL: Make FUSE available in release builds #135603
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
Conversation
Hi @ioanatia, I've created a changelog YAML for you. |
Pinging @elastic/es-search-relevance (Team:Search Relevance) |
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.
LGTM
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.
LGTM. Nice adding the L2_NORM as a different capability.
In this PR or in a follow up, we can clean up the references to FUSE_V6 capability in the Analyzer, StatementParser, Verifier, FieldNameUtils tests, as it will be available both in release and snapshot builds and are unnecessary.
tracked in #123389
One thing to note is that L2_NORM score normalization support will continue to be in snapshot.
We still support minmax as a score normalization method which will be the most used.
The reason why we keep L2_NORM in snapshot is because we still need to decide on what would be the best behaviour when the scores are negative (which is not a problem we have with DSL).