Skip to content
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

Wrong fields used in the filter and aggregation #126

Closed
liu-xiao-guo opened this issue Nov 28, 2023 · 4 comments · Fixed by #339
Closed

Wrong fields used in the filter and aggregation #126

liu-xiao-guo opened this issue Nov 28, 2023 · 4 comments · Fixed by #339
Assignees
Labels
bug Something isn't working

Comments

@liu-xiao-guo
Copy link

For the notebook at https://github.com/elastic/elasticsearch-labs/blob/main/supporting-blog-content/vector-search-implementation-guide-api/vector_search_implementation_guide_api.ipynb, "my_metadata" is a text field, and it should not be used in the term filter, and it is the same for the aggregation. The correct one should be "my_metadata.keyword".

image
image

@miguelgrinberg
Copy link
Collaborator

@jeffvestal Will you be looking at this? The corresponding blog article needs to be fixed as well, it appears.

@serenachou serenachou added the bug Something isn't working label Feb 8, 2024
@serenachou
Copy link
Collaborator

@jeffvestal just a ping on this one

@jeffvestal
Copy link
Contributor

Hey 11 months later
Sorry, this slipped way by me

fixed in #339

@ElishevaStern
Copy link

Thanks @jeffvestal !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants