Skip to content

Conversation

@salvatore-campagna
Copy link
Contributor

@salvatore-campagna salvatore-campagna commented Apr 26, 2023

Here we test the bahaviour of downsampling with missing or null labels.
The test shows that we always keep the most recent non-null non-missing
value for a specific fixed interval.

@salvatore-campagna salvatore-campagna self-assigned this Apr 26, 2023
@salvatore-campagna salvatore-campagna added :StorageEngine/Rollup Turn fine-grained time-based data into coarser-grained data >test Issues or PRs that are addressing/adding tests Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) labels Apr 26, 2023
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytics-geo (Team:Analytics)

Copy link
Member

@martijnvg martijnvg left a comment

Choose a reason for hiding this comment

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

One comment, otherwise LGTM.

---
"Downsample empty and missing labels":
- skip:
version: " - 8.4.99"
Copy link
Member

Choose a reason for hiding this comment

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

should this be 8.6.99 ?

Copy link
Contributor Author

@salvatore-campagna salvatore-campagna Apr 26, 2023

Choose a reason for hiding this comment

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

I always try to put there the minimum possible value which is 8.4.99 because in 8.5.0 we changed something around naming which also includes renaming endpoints. My understanding is that this (using the minimum possible versions) allows us to run those tests also in mixed cluster tests where there are nodes running older version of ES. Am I wrong?

This should indeed work also for versions 8.5, 8.6 and 8.7.

Copy link
Member

Choose a reason for hiding this comment

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

If from 8.5.0 there are no breaking changes then this skip version makes sense.

There is another aspect and that prior to 8.7.0, downsampling was tech preview.
With that in mind, I think actually many of our tsdb yaml tests can skip versions older than that version.
But maybe we can chat about this next week.

Use the Downsampling GA version as the minimum required.
@salvatore-campagna
Copy link
Contributor Author

@elasticsearchmachine test this please

@salvatore-campagna salvatore-campagna merged commit 086993c into elastic:main May 3, 2023
@gmarouli gmarouli added v8.8.0 and removed v8.8.1 labels May 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:StorageEngine/Rollup Turn fine-grained time-based data into coarser-grained data Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v8.8.0 v8.9.0

4 participants