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

Rename AggregateDoubleMetric to *MetricDouble #121254

Conversation

limotova
Copy link
Contributor

Some areas of the code call this field type
AggregateDoubleMetric and others AggregateMetricDouble, but the docs
use aggregate_metric_double, so for consistency this commit refactors
the former into the latter.

Some areas of the code call this field type
AggregateDoubleMetric and others AggregateMetricDouble, but the docs
use aggregate_metric_double, so for consistency this commit refactors
the former into the latter.
@limotova limotova force-pushed the refactor-aggregate-metric-double-for-consistency branch from cfb4fc6 to d50dca1 Compare January 31, 2025 18:16
@limotova limotova requested a review from martijnvg February 1, 2025 00:42
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.

LGTM 👍

Let's also backport this to 8.x, 9.0 and 8.18 branch? This makes back porting easier.

@limotova limotova marked this pull request as ready for review February 4, 2025 02:01
@limotova limotova added the auto-backport Automatically create backport pull requests when merged label Feb 4, 2025
@elasticsearchmachine elasticsearchmachine added the needs:triage Requires assignment of a team area label label Feb 4, 2025
@martijnvg martijnvg added >non-issue :StorageEngine/Downsampling Downsampling (replacement for rollups) - Turn fine-grained time-based data into coarser-grained data v9.0.0 and removed needs:triage Requires assignment of a team area label labels Feb 4, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@limotova limotova merged commit af213a5 into elastic:main Feb 4, 2025
17 checks passed
@limotova limotova deleted the refactor-aggregate-metric-double-for-consistency branch February 4, 2025 18:56
limotova added a commit to limotova/elasticsearch that referenced this pull request Feb 4, 2025
Some areas of the code call this field type
AggregateDoubleMetric and others AggregateMetricDouble, but the docs
use aggregate_metric_double, so for consistency this commit refactors
the former into the latter.
limotova added a commit to limotova/elasticsearch that referenced this pull request Feb 4, 2025
Some areas of the code call this field type
AggregateDoubleMetric and others AggregateMetricDouble, but the docs
use aggregate_metric_double, so for consistency this commit refactors
the former into the latter.
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
9.0
8.18
8.x

elasticsearchmachine pushed a commit that referenced this pull request Feb 4, 2025
Some areas of the code call this field type
AggregateDoubleMetric and others AggregateMetricDouble, but the docs
use aggregate_metric_double, so for consistency this commit refactors
the former into the latter.
elasticsearchmachine pushed a commit that referenced this pull request Feb 4, 2025
Some areas of the code call this field type
AggregateDoubleMetric and others AggregateMetricDouble, but the docs
use aggregate_metric_double, so for consistency this commit refactors
the former into the latter.
elasticsearchmachine pushed a commit that referenced this pull request Feb 4, 2025
Some areas of the code call this field type
AggregateDoubleMetric and others AggregateMetricDouble, but the docs
use aggregate_metric_double, so for consistency this commit refactors
the former into the latter.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Automatically create backport pull requests when merged >non-issue :StorageEngine/Downsampling Downsampling (replacement for rollups) - Turn fine-grained time-based data into coarser-grained data Team:StorageEngine v8.18.1 v8.19.0 v9.0.0 v9.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants