Skip to content

Add sparse_vector field type#2304

Merged
pquentin merged 2 commits intomainfrom
carlosdelest/add_sparse_vector_field_type
Oct 6, 2023
Merged

Add sparse_vector field type#2304
pquentin merged 2 commits intomainfrom
carlosdelest/add_sparse_vector_field_type

Conversation

@carlosdelest
Copy link
Member

Add sparse_vector field support (added to Elasticsearch elastic/elasticsearch#98996) to mappings, and include it as usable in the docs for text_expansion query.

Copy link
Member

@pquentin pquentin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! LGTM.

By the way I think the documentation is missing for that field in the Elasticsearch docs. I can see rank_feature but not sparse_vector: https://www.elastic.co/guide/en/elasticsearch/reference/master/rank-feature.html

@pquentin pquentin merged commit e34cdb6 into main Oct 6, 2023
@pquentin pquentin deleted the carlosdelest/add_sparse_vector_field_type branch October 6, 2023 10:34
github-actions bot pushed a commit that referenced this pull request Oct 6, 2023
pquentin pushed a commit that referenced this pull request Oct 6, 2023
* Add sparse_vector field type (#2304)

(cherry picked from commit e34cdb6)

---------

Co-authored-by: Carlos Delgado <6339205+carlosdelest@users.noreply.github.com>
@carlosdelest
Copy link
Member Author

I think the documentation is missing for that field in the Elasticsearch docs

Thanks for looking into that @pquentin ! It's just been merged in elastic/elasticsearch#100348 👍

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants