Skip to content

[8.18] Avoid reading unnecessary dimension values when downsampling (#124451)#124468

Merged
elasticsearchmachine merged 1 commit intoelastic:8.18from
martijnvg:backport/8.18/pr-124451
Mar 10, 2025
Merged

[8.18] Avoid reading unnecessary dimension values when downsampling (#124451)#124468
elasticsearchmachine merged 1 commit intoelastic:8.18from
martijnvg:backport/8.18/pr-124451

Conversation

@martijnvg
Copy link
Member

Backports the following commits to 8.18:

…#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 ad6cbfa into elastic:8.18 Mar 10, 2025
15 checks passed
@martijnvg martijnvg deleted the backport/8.18/pr-124451 branch March 10, 2025 12:19
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.18.1

2 participants