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

Jump to bottom

Optimize global ordinal includes/excludes for prefix matching #14371

Merged
merged 7 commits into from
Aug 20, 2024

Address comments from @mch2

1bc728a
Select commit
Loading
Failed to load commit list.
Merged

Optimize global ordinal includes/excludes for prefix matching #14371

Address comments from @mch2
1bc728a
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Aug 20, 2024 in 0s

77.88% of diff hit (target 71.95%)

View this Pull Request on Codecov

77.88% of diff hit (target 71.95%)

Annotations

Check warning on line 421 in server/src/main/java/org/opensearch/search/aggregations/bucket/terms/IncludeExclude.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/search/aggregations/bucket/terms/IncludeExclude.java#L420-L421 

Added lines #L420 - L421 were not covered by tests

Check warning on line 427 in server/src/main/java/org/opensearch/search/aggregations/bucket/terms/IncludeExclude.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/search/aggregations/bucket/terms/IncludeExclude.java#L427 

Added line #L427 was not covered by tests

Check warning on line 454 in server/src/main/java/org/opensearch/search/aggregations/bucket/terms/IncludeExclude.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/search/aggregations/bucket/terms/IncludeExclude.java#L454 

Added line #L454 was not covered by tests

Check warning on line 458 in server/src/main/java/org/opensearch/search/aggregations/bucket/terms/IncludeExclude.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/search/aggregations/bucket/terms/IncludeExclude.java#L458 

Added line #L458 was not covered by tests

Check warning on line 462 in server/src/main/java/org/opensearch/search/aggregations/bucket/terms/IncludeExclude.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/search/aggregations/bucket/terms/IncludeExclude.java#L462 

Added line #L462 was not covered by tests

Check warning on line 901 in server/src/main/java/org/opensearch/search/aggregations/bucket/terms/IncludeExclude.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/search/aggregations/bucket/terms/IncludeExclude.java#L901 

Added line #L901 was not covered by tests

Check warning on line 905 in server/src/main/java/org/opensearch/search/aggregations/bucket/terms/IncludeExclude.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/search/aggregations/bucket/terms/IncludeExclude.java#L905 

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