Skip to content

Ensure we parse on_disk_rescore as its valid to pass false#141158

Merged
elasticsearchmachine merged 3 commits intoelastic:mainfrom
benwtrent:feature/parse-mappings-dense-vectors
Jan 23, 2026
Merged

Ensure we parse on_disk_rescore as its valid to pass false#141158
elasticsearchmachine merged 3 commits intoelastic:mainfrom
benwtrent:feature/parse-mappings-dense-vectors

Conversation

@benwtrent
Copy link
Member

its valid to pass false for on_disk_rescore for all index mapping types. We should parse the key to validate.

I noticed this when changing combinations in our rally tracks :) (which defaults to sending a false value).

@benwtrent benwtrent requested a review from thecoop January 23, 2026 11:57
@benwtrent benwtrent added >bug auto-backport Automatically create backport pull requests when merged :Search Relevance/Vectors Vector search v9.3.1 v9.4.0 labels Jan 23, 2026
@elasticsearchmachine elasticsearchmachine added the Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch label Jan 23, 2026
@elasticsearchmachine
Copy link
Collaborator

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

Copy link
Member

@thecoop thecoop left a comment

Choose a reason for hiding this comment

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

Minor tweak, LGTM

…nseVectorFieldMapper.java

Co-authored-by: Simon Cooper <simon.cooper@elastic.co>
@benwtrent benwtrent added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Jan 23, 2026
@elasticsearchmachine
Copy link
Collaborator

Hi @benwtrent, I've created a changelog YAML for you.

@elasticsearchmachine elasticsearchmachine merged commit 14b8ff3 into elastic:main Jan 23, 2026
35 checks passed
@benwtrent benwtrent deleted the feature/parse-mappings-dense-vectors branch January 23, 2026 15:35
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
9.3
benwtrent added a commit to benwtrent/elasticsearch that referenced this pull request Jan 23, 2026
…41158)

its valid to pass false for on_disk_rescore for all index mapping types.
We should parse the key to validate.

I noticed this when changing combinations in our rally tracks :) (which
defaults to sending a false value).
elasticsearchmachine pushed a commit that referenced this pull request Jan 23, 2026
…141191)

its valid to pass false for on_disk_rescore for all index mapping types.
We should parse the key to validate.

I noticed this when changing combinations in our rally tracks :) (which
defaults to sending a false value).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) >bug :Search Relevance/Vectors Vector search Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch v9.3.1 v9.4.0

3 participants