Skip to content

Conversation

@weizijun
Copy link
Contributor

The semantic_text type is very useful for the AI Search.
When inference_id is text_embedding, the semantic_text internal field is dense_vector type.
But now the semantic_text can't support index_options of the dense_vector type.
User can't set quantification type or m, ef parameters.
So I added the index_options in the semantic_text type.

@elasticsearchmachine elasticsearchmachine added needs:triage Requires assignment of a team area label v9.1.0 external-contributor Pull request authored by a developer outside the Elasticsearch team labels Feb 20, 2025
@PeteGillinElastic PeteGillinElastic added :SearchOrg/Relevance Label for the Search (solution/org) Relevance team and removed needs:triage Requires assignment of a team area label labels Feb 21, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/search-eng (Team:SearchOrg)

@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/search-relevance (Team:Search - Relevance)

@jimczi
Copy link
Contributor

jimczi commented Mar 7, 2025

Thanks, @weizijun, and apologies for the delayed response.

We're actively working on adding index options to the semantic text field in [this PR](#119967). During this process, we realized that validating these index options requires an underlying infrastructure change, which we're addressing in [this PR](#121106). This additional work has caused some delays.

Once the infrastructure change is merged, we'll update the original PR to incorporate the necessary logic, ensuring that invalid index options are detected and rejected early.

@weizijun
Copy link
Contributor Author

Thanks, @weizijun, and apologies for the delayed response.

We're actively working on adding index options to the semantic text field in [this PR](#119967). During this process, we realized that validating these index options requires an underlying infrastructure change, which we're addressing in [this PR](#121106). This additional work has caused some delays.

Once the infrastructure change is merged, we'll update the original PR to incorporate the necessary logic, ensuring that invalid index options are detected and rejected early.

Thanks @jimczi , what a coincidence, this feature is really useful and I hope it can be merged soon.

@kderusso
Copy link
Member

Hi @weizijun , thanks for the work on this PR. I'm going to close it, as we've merged a separate feature that will address this need in an upcoming release: #119967

This will allow you to set index_options for dense vector fields.

@kderusso kderusso closed this Jun 18, 2025
@weizijun
Copy link
Contributor Author

Hi @weizijun , thanks for the work on this PR. I'm going to close it, as we've merged a separate feature that will address this need in an upcoming release: #119967

Thanks @kderusso !

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

Labels

external-contributor Pull request authored by a developer outside the Elasticsearch team :SearchOrg/Relevance Label for the Search (solution/org) Relevance team v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants