Skip to content

[TEST] Skip injecting boundary tuple only on lower boundary for rate tests#140857

Merged
kkrik-es merged 2 commits intoelastic:mainfrom
kkrik-es:fix/140853
Jan 19, 2026
Merged

[TEST] Skip injecting boundary tuple only on lower boundary for rate tests#140857
kkrik-es merged 2 commits intoelastic:mainfrom
kkrik-es:fix/140853

Conversation

@kkrik-es
Copy link
Copy Markdown
Member

Follow up on #140649, we still need to inject a tuple on the upper boundary if applicable.

Fixes #139639
Fixes #140853

@kkrik-es kkrik-es self-assigned this Jan 18, 2026
@kkrik-es kkrik-es added >test Issues or PRs that are addressing/adding tests auto-backport Automatically create backport pull requests when merged Team:StorageEngine :StorageEngine/ES|QL Timeseries / metrics / PromQL / logsdb capabilities in ES|QL v9.3.1 labels Jan 18, 2026
@kkrik-es kkrik-es requested a review from dnhatn January 18, 2026 16:02
@kkrik-es kkrik-es marked this pull request as ready for review January 18, 2026 16:02
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-storage-engine (Team:StorageEngine)

Copy link
Copy Markdown
Member

@dnhatn dnhatn left a comment

Choose a reason for hiding this comment

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

LGTM - thanks for fixing this, Kostas!

@kkrik-es kkrik-es merged commit 1d24a8d into elastic:main Jan 19, 2026
35 checks passed
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

💔 Backport failed

Status Branch Result
9.3 Commit could not be cherrypicked due to conflicts

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 140857

@kkrik-es
Copy link
Copy Markdown
Member Author

💚 All backports created successfully

Status Branch Result
9.3

Questions ?

Please refer to the Backport tool documentation

kkrik-es added a commit to kkrik-es/elasticsearch that referenced this pull request Jan 19, 2026
…tests (elastic#140857)

* [TEST] Skip injecting boundary tuple only on lower boundary for rate tests

* [CI] Auto commit changes from spotless

---------

Co-authored-by: elasticsearchmachine <infra-root+elasticsearchmachine@elastic.co>
(cherry picked from commit 1d24a8d)

# Conflicts:
#	muted-tests.yml
elasticsearchmachine pushed a commit that referenced this pull request Jan 19, 2026
… rate tests (#140857) (#140873)

* [TEST] Skip injecting boundary tuple only on lower boundary for rate tests (#140857)

* [TEST] Skip injecting boundary tuple only on lower boundary for rate tests

* [CI] Auto commit changes from spotless

---------

Co-authored-by: elasticsearchmachine <infra-root+elasticsearchmachine@elastic.co>
(cherry picked from commit 1d24a8d)

# Conflicts:
#	muted-tests.yml

* Remove muted tests and add new test case
@kkrik-es kkrik-es deleted the fix/140853 branch January 19, 2026 11:36
spinscale pushed a commit to spinscale/elasticsearch that referenced this pull request Jan 21, 2026
…tests (elastic#140857)

* [TEST] Skip injecting boundary tuple only on lower boundary for rate tests

* [CI] Auto commit changes from spotless

---------

Co-authored-by: elasticsearchmachine <infra-root+elasticsearchmachine@elastic.co>
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 backport pending :StorageEngine/ES|QL Timeseries / metrics / PromQL / logsdb capabilities in ES|QL Team:StorageEngine >test Issues or PRs that are addressing/adding tests v9.3.1 v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] RandomizedTimeSeriesIT testRateGroupBySubset failing [CI] RandomizedTimeSeriesIT testRateGroupByNothing failing

3 participants