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

Support Dynamic Pruning in Cardinality Aggregation #13821

Merged
merged 30 commits into from
Jun 11, 2024

address comments

5f7dad3
Select commit
Loading
Failed to load commit list.
Merged

Support Dynamic Pruning in Cardinality Aggregation #13821

address comments
5f7dad3
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Jun 11, 2024 in 0s

78.18% of diff hit (target 71.42%)

View this Pull Request on Codecov

78.18% of diff hit (target 71.42%)

Annotations

Check warning on line 1019 in server/src/main/java/org/opensearch/search/DefaultSearchContext.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/search/DefaultSearchContext.java#L1019 

Added line #L1019 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/search/aggregations/metrics/CardinalityAggregator.java#L162 

Added line #L162 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/search/aggregations/metrics/CardinalityAggregator.java#L181 

Added line #L181 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/search/aggregations/metrics/CardinalityAggregator.java#L185 

Added line #L185 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/search/aggregations/metrics/CardinalityAggregator.java#L215-L217 

Added lines #L215 - L217 were not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/search/aggregations/metrics/CardinalityAggregator.java#L226 

Added line #L226 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/search/aggregations/metrics/CardinalityAggregator.java#L232-L233 

Added lines #L232 - L233 were not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/search/aggregations/metrics/CardinalityAggregator.java#L235 

Added line #L235 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/search/aggregations/metrics/CardinalityAggregator.java#L317 

Added line #L317 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/search/aggregations/metrics/CardinalityAggregator.java#L423 

Added line #L423 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/search/aggregations/metrics/CardinalityAggregator.java#L445 

Added line #L445 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/search/aggregations/metrics/CardinalityAggregator.java#L451 

Added line #L451 was not covered by tests

Check warning on line 527 in server/src/main/java/org/opensearch/search/internal/SearchContext.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/search/internal/SearchContext.java#L527 

Added line #L527 was not covered by tests
View more details on Codecov