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

Add cluster setting to dynamically disable filter rewrite optimization #13179

Merged

Address comments

4bded8f
Select commit
Loading
Failed to load commit list.
Merged

Add cluster setting to dynamically disable filter rewrite optimization #13179

Address comments
4bded8f
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Apr 16, 2024 in 1s

45.45% of diff hit (target 71.42%)

View this Pull Request on Codecov

45.45% of diff hit (target 71.42%)

Annotations

Check warning on line 1004 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#L1004

Added line #L1004 was not covered by tests

Check warning on line 188 in server/src/main/java/org/opensearch/search/aggregations/bucket/FastFilterRewriteHelper.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/search/aggregations/bucket/FastFilterRewriteHelper.java#L188

Added line #L188 was not covered by tests

Check warning on line 190 in server/src/main/java/org/opensearch/search/aggregations/bucket/FastFilterRewriteHelper.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/search/aggregations/bucket/FastFilterRewriteHelper.java#L190

Added line #L190 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