Skip to content

Conversation

@limotova
Copy link
Contributor

@limotova limotova commented Aug 1, 2025

Aggregate Metric Double was missing from quite a few of these tests

@limotova limotova force-pushed the agg-metric-block-tests branch 9 times, most recently from 5031978 to d3b8e6d Compare August 26, 2025 00:09
@limotova limotova force-pushed the agg-metric-block-tests branch from d3b8e6d to 6d2bf92 Compare August 26, 2025 01:06
@limotova limotova changed the title add aggregate metric double block to other tests [ES|QL] Add more aggregate metric double tests Aug 26, 2025
@limotova
Copy link
Contributor Author

Relates to #131951

Copy link
Contributor

@kkrik-es kkrik-es left a comment

Choose a reason for hiding this comment

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

Nice.

Copy link
Member

@not-napoleon not-napoleon left a comment

Choose a reason for hiding this comment

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

LGTM

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.

LGTM, thanks Larisa!

@limotova limotova added >test Issues or PRs that are addressing/adding tests :StorageEngine/TSDB You know, for Metrics :StorageEngine/ES|QL Timeseries / metrics / logsdb capabilities in ES|QL labels Aug 26, 2025
@limotova limotova marked this pull request as ready for review August 26, 2025 20:28
@elasticsearchmachine
Copy link
Collaborator

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

@limotova limotova merged commit 04ed058 into elastic:main Aug 26, 2025
33 checks passed
@limotova limotova deleted the agg-metric-block-tests branch August 26, 2025 21:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:StorageEngine/ES|QL Timeseries / metrics / logsdb capabilities in ES|QL :StorageEngine/TSDB You know, for Metrics Team:StorageEngine >test Issues or PRs that are addressing/adding tests v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants