-
Notifications
You must be signed in to change notification settings - Fork 24.9k
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
Add some docs explaining filter performance and behavior for HNSW #110108
Add some docs explaining filter performance and behavior for HNSW #110108
Conversation
Documentation preview: |
Pinging @elastic/es-docs (Team:Docs) |
Pinging @elastic/es-search (Team:Search) |
Backporting to 8.4 as that is when knn was GA'd |
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.
Nice addition, thanks Ben!
We should explain how HNSW can be slower with filtering and that there are two mitigations in place to address this.
This is not meant to be exhaustive with benchmarks, etc. I think a larger blog post is warranted for overall performance with filtering and explaining how HNSW graph exploration is effected.