Skip to content

Conversation

@pabloem
Copy link
Contributor

@pabloem pabloem commented Oct 3, 2025

Fixes #135756 #135757 #135814 #135775 #135755 #135753 #135754

The algorithm to calculate outcome was improved, but it also gives a lot of margin for error because correctness for all of these is verified in RandomizedTimeSeriesIT.

@elasticsearchmachine elasticsearchmachine added needs:triage Requires assignment of a team area label v9.3.0 labels Oct 3, 2025
@pabloem pabloem added >test Issues or PRs that are addressing/adding tests :StorageEngine/TSDB You know, for Metrics Team:StorageEngine and removed needs:triage Requires assignment of a team area label labels Oct 3, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@kkrik-es kkrik-es requested review from dnhatn and removed request for not-napoleon October 7, 2025 06:08
@pabloem pabloem requested a review from kkrik-es October 7, 2025 16:14
Copy link
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.

Thanks Pablo!

@pabloem pabloem merged commit f5689af into elastic:main Oct 8, 2025
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:StorageEngine/TSDB You know, for Metrics Team:StorageEngine >test Issues or PRs that are addressing/adding tests v9.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] IrateTests testGroupingAggregate {TestCase=<big positive doubles>} failing

4 participants