Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

default_metric field is not propagated when downsampling an aggregate_metric_double field #119696

Open
lkts opened this issue Jan 7, 2025 · 2 comments
Labels
>bug :StorageEngine/Downsampling Downsampling (replacement for rollups) - Turn fine-grained time-based data into coarser-grained data Team:StorageEngine

Comments

@lkts
Copy link
Contributor

lkts commented Jan 7, 2025

Elasticsearch Version

9.0 and before

Installed Plugins

No response

Java Version

bundled

OS Version

Problem Description

default_metric mapping parameter is lost during downsampling.

Steps to Reproduce

Downsample an index with the following mapping:

"m": {
  "type": "aggregate_metric_double",
  "metrics": ["min", "max", "sum", "value_count"],
  "default_metric": "min",
  "time_series_metric": "gauge"
}

Downsampled index has the following mapping:

"m": {
    "type": "aggregate_metric_double",
    "metrics": [
        "min",
        "max",
        "sum",
        "value_count"
    ],
    "default_metric": "max",
    "time_series_metric": "gauge"
}

Note the change of default_metric from min to max.

Logs (if relevant)

No response

@lkts lkts added :StorageEngine/Downsampling Downsampling (replacement for rollups) - Turn fine-grained time-based data into coarser-grained data >bug labels Jan 7, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@lkts
Copy link
Contributor Author

lkts commented Jan 7, 2025

It happens here

It seems that this code needs explicit handling of aggregate_metric_double?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug :StorageEngine/Downsampling Downsampling (replacement for rollups) - Turn fine-grained time-based data into coarser-grained data Team:StorageEngine
Projects
None yet
Development

No branches or pull requests

2 participants