-
Notifications
You must be signed in to change notification settings - Fork 25.6k
Mark bbq indices as GA and add rolling upgrade integration tests #121105
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
|
Documentation preview: |
|
Pinging @elastic/es-search-relevance (Team:Search Relevance) |
|
Hi @benwtrent, I've created a changelog YAML for you. |
pmpailis
left a comment
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.
🚀
|
|
||
| NOTE: `int4` quantization requires an even number of vector dimensions. | ||
|
|
||
| NOTE: experimental:[] `bbq` quantization only supports vector dimensions that are greater than 64. |
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.
missed an experimental marker on line 180
john-wagster
left a comment
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.
other than the 1 comment i left this LGTM
carlosdelest
left a comment
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 LGTM.
I guess rescore_vector should be kept as tech preview, correct? Otherwise we need to change that in the docs
💚 Backport successful
|
With the introduction of our new backing algorithm and making rescoring easier with the
rescore_vectorAPI, let's mark bbq as GA.Additionally, this commit adds rolling upgrade tests to ensure stability.