Open wirybeaver opened 1 week ago
Attention: Patch coverage is 95.55556%
with 2 lines
in your changes are missing coverage. Please review.
Project coverage is 62.17%. Comparing base (
59551e4
) to head (60a8cbf
). Report is 453 commits behind head on master.
Files | Patch % | Lines |
---|---|---|
...t/core/operator/filter/FilterOperatorRewriter.java | 95.00% | 1 Missing and 1 partial :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
The issue https://github.com/apache/pinot/issues/13081 says it at all. We should have recursively visited the child operators of AND / OR operator and use the lowest priority (largest value) of children as the priority of the current AND / OR operator. For AND's kids operator, reorder based on the priority.