Skip to content

[Backport 2.x] [bug fix] fix precompute ordering in max & min aggregator - #17646

Closed
opensearch-trigger-bot[bot] wants to merge 1 commit into
2.xfrom
backport/backport-17505-to-2.x
Closed

[Backport 2.x] [bug fix] fix precompute ordering in max & min aggregator#17646
opensearch-trigger-bot[bot] wants to merge 1 commit into
2.xfrom
backport/backport-17505-to-2.x

fix precompute ordering for min & max aggregators (#17505)

753ccef
Select commit
Loading
Failed to load commit list.
This check has been archived and is scheduled for deletion. Learn more about checks retention
Codecov / codecov/patch failed Jun 23, 2025 in 1s

26.66% of diff hit (target 72.02%)

View this Pull Request on Codecov

26.66% of diff hit (target 72.02%)

Annotations

Check warning on line 123 in server/src/main/java/org/opensearch/search/aggregations/metrics/MaxAggregator.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/search/aggregations/metrics/MaxAggregator.java#L121-L123

Added lines #L121 - L123 were not covered by tests

Check warning on line 125 in server/src/main/java/org/opensearch/search/aggregations/metrics/MaxAggregator.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/search/aggregations/metrics/MaxAggregator.java#L125

Added line #L125 was not covered by tests

Check warning on line 122 in server/src/main/java/org/opensearch/search/aggregations/metrics/MinAggregator.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/search/aggregations/metrics/MinAggregator.java#L120-L122

Added lines #L120 - L122 were not covered by tests

Check warning on line 124 in server/src/main/java/org/opensearch/search/aggregations/metrics/MinAggregator.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/search/aggregations/metrics/MinAggregator.java#L124

Added line #L124 was not covered by tests