Skip to content

[9.3] fix(esql): check per-shard DateFieldType for DocValuesSkipper (#142752)#142800

Merged
elasticsearchmachine merged 1 commit intoelastic:9.3from
salvatore-campagna:backport/9.3/pr-142752
Feb 21, 2026
Merged

[9.3] fix(esql): check per-shard DateFieldType for DocValuesSkipper (#142752)#142800
elasticsearchmachine merged 1 commit intoelastic:9.3from
salvatore-campagna:backport/9.3/pr-142752

Conversation

@salvatore-campagna
Copy link
Contributor

Backports the following commits to 9.3:

…c#142752)

* fix(esql): check per-shard DateFieldType for DocValuesSkipper in SearchContextStats min/max

hasDocValuesSkipper was derived once from the first shard's
DateFieldType and applied globally via doWithContexts. In mixed
environments (TSDB shards with DocValuesSkipper + standard shards
with PointValues), calling the wrong API on some shards caused
sentinel values (Long.MIN_VALUE/Long.MAX_VALUE) to leak into min/max.

Replace doWithContexts with direct per-context iteration so each
shard's own DateFieldType determines whether to use DocValuesSkipper
or PointValues. This also simplifies the early-return guard by
removing the incorrect indexed check that could bail out prematurely
in mixed modes.
@salvatore-campagna salvatore-campagna added :StorageEngine/TSDB You know, for Metrics >non-issue auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport backport pending Team:StorageEngine labels Feb 21, 2026
@elasticsearchmachine elasticsearchmachine merged commit 8d3b485 into elastic:9.3 Feb 21, 2026
35 checks passed
@salvatore-campagna salvatore-campagna deleted the backport/9.3/pr-142752 branch February 21, 2026 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport pending backport >non-issue :StorageEngine/TSDB You know, for Metrics Team:StorageEngine v9.3.2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants