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.

Sign up for GitHub

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

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

Open
wants to merge 1 commit into
base: 2.x
Choose a base branch
from
+33 −29

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

753ccef
Select commit
Loading
Failed to load commit list.
Open

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

fix precompute ordering for min & max aggregators (#17505)
753ccef
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Mar 20, 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
View more details on Codecov