Skip to content

Missing 'similarity' field in knn search request #2149

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

Closed
adrian-arapiles opened this issue Jun 20, 2023 · 5 comments
Closed

Missing 'similarity' field in knn search request #2149

adrian-arapiles opened this issue Jun 20, 2023 · 5 comments

Comments

@adrian-arapiles
Copy link

From 8.8 version, elasticsearch has field 'similarity' for search in knn field.
Right now, the api doesn't support this field and fail.

🐛 Missing type

https://www.elastic.co/guide/en/elasticsearch/reference/current/knn-search.html#_search_knn_with_expected_similarity
Unknown field 'similarity' (JSON path: similarity) (line no=2, column no=15, offset=16)

Definition

If possible provide a basic definition that we can use as base.
You can see here how to create a basic definition for any API.

@adrian-arapiles
Copy link
Author

I tried to push a branch with the change, but I haven't permissions.

@pstrsr
Copy link
Contributor

pstrsr commented Aug 2, 2023

elastic/elasticsearch-java#643 Related.

I also tried to open a PR with the fix, but same issue.

@ezimuel
Copy link
Contributor

ezimuel commented Aug 3, 2023

@adrian-arapiles did you fork the project and send a PR from a branch in your fork? You cannot create a branch in the original repository. /cc @AlmostFamiliar

@pstrsr
Copy link
Contributor

pstrsr commented Aug 3, 2023

@ezimuel that worked. I have never tried making a PR for an OS project, thank you for helping out!

@JoshMock
Copy link
Member

#2261 is merged and backported to 8.8, 8.9 and 8.10, so this should be fixed in the next patch release for each of those, or 8.11.0 when it releases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants