Skip to content

[TEST] Add rounding to time series agg results#143805

Merged
dnhatn merged 1 commit intoelastic:mainfrom
kkrik-es:fix/143757
Mar 8, 2026
Merged

[TEST] Add rounding to time series agg results#143805
dnhatn merged 1 commit intoelastic:mainfrom
kkrik-es:fix/143757

Conversation

@kkrik-es
Copy link
Copy Markdown
Member

@kkrik-es kkrik-es commented Mar 8, 2026

CsvIT doesn't use force merging so there may be small differences in results due to float processing.

Unmuting also some tests that were fixed earlier but #143657 muted again, likely due to a merge conflict.

Fixes #143758
Fixes #143757

@kkrik-es kkrik-es requested a review from dnhatn March 8, 2026 14:06
@kkrik-es kkrik-es self-assigned this Mar 8, 2026
@kkrik-es kkrik-es added >test Issues or PRs that are addressing/adding tests Team:StorageEngine :StorageEngine/ES|QL Timeseries / metrics / PromQL / logsdb capabilities in ES|QL labels Mar 8, 2026
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.

Thanks Kostas!

@dnhatn dnhatn marked this pull request as ready for review March 8, 2026 17:02
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

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

@dnhatn dnhatn merged commit d10b557 into elastic:main Mar 8, 2026
35 of 36 checks passed
@kkrik-es kkrik-es deleted the fix/143757 branch March 9, 2026 07:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:StorageEngine/ES|QL Timeseries / metrics / PromQL / logsdb capabilities in ES|QL Team:StorageEngine >test Issues or PRs that are addressing/adding tests v9.4.0

Projects

None yet

3 participants