Skip to content

[8.x] Avoid reading unnecessary dimension values when downsampling (#124451)#124469

Merged
elasticsearchmachine merged 1 commit intoelastic:8.xfrom
martijnvg:backport/8.x/pr-124451
Mar 10, 2025
Merged

[8.x] Avoid reading unnecessary dimension values when downsampling (#124451)#124469
elasticsearchmachine merged 1 commit intoelastic:8.xfrom
martijnvg:backport/8.x/pr-124451

Conversation

@martijnvg
Copy link
Member

Backports the following commits to 8.x:

…#124451)

Read dimension values once per tsid/bucket docid range instead of for each document being processed.
The dimension value within a bucket-interval docid range is always to same and this avoids unnecessary reads.

Latency of downsampling the tsdb track index into a 1 hour interval downsample index drop by ~16% (running on my local machine).
@martijnvg martijnvg added :StorageEngine/Downsampling Downsampling (replacement for rollups) - Turn fine-grained time-based data into coarser-grained data >bug auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:StorageEngine labels Mar 10, 2025
@elasticsearchmachine elasticsearchmachine merged commit fe98c05 into elastic:8.x Mar 10, 2025
15 checks passed
@martijnvg martijnvg deleted the backport/8.x/pr-124451 branch March 10, 2025 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport >bug :StorageEngine/Downsampling Downsampling (replacement for rollups) - Turn fine-grained time-based data into coarser-grained data Team:StorageEngine v8.19.0

2 participants