Introduce OR Predicate Execution On Star Tree Index

[atris]

This commit allows OR predicates to be executed on Star Tree indices. We only allow a single dimension to
be present in the OR predicate in order to be qualified for execution on Star Tree node. This is to ensure
that we do not double count pre aggregated documents while traversing the tree for two dimensions. A follow
up PR will be done for multi dimensional OR predicates.

[Jackie-Jiang]

The filter we want to support here could be nested OR under AND, e.g. a > 10 AND (b < 0 OR b > 10) AND (c < 5 OR c > 20). We should not assume the filter only contain one of AND and OR

[atris]

The integration test is failing due to a Kafka issue -- unrelated IMO to this change

[atris]

@Jackie-Jiang Updated, please see

[Jackie-Jiang]

@atris The test failure is in star-tree cluster integration test, which is related to the change in this PR. PTAL

[atris]

@atris The test failure is in star-tree cluster integration test, which is related to the change in this PR. PTAL

Thanks, fixed.

@Jackie-Jiang Updated the PR, please see and let me know your comments and feedback.

[codecov-commenter]

Codecov Report

Merging #7184 (4a51773) into master (fe83e95) will decrease coverage by 8.24%.
The diff coverage is 68.45%.

❗ Current head 4a51773 differs from pull request most recent head f7d6bf4. Consider uploading reports for the commit f7d6bf4 to get more accurate results

@@             Coverage Diff              @@
##             master    #7184      +/-   ##
============================================
- Coverage     73.51%   65.27%   -8.25%     
  Complexity       92       92              
============================================
  Files          1506     1508       +2     
  Lines         73832    73939     +107     
  Branches      10655    10682      +27     
============================================
- Hits          54281    48262    -6019     
- Misses        16011    22260    +6249     
+ Partials       3540     3417     -123     

Flag	Coverage Δ
integration	?
unittests	65.27% <68.45%> (+0.02%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
.../main/java/org/apache/pinot/client/Connection.java	34.04% <0.00%> (-9.44%)	⬇️
...n/java/org/apache/pinot/client/ExecutionStats.java	68.88% <ø> (ø)
.../org/apache/pinot/client/ResultTableResultSet.java	88.00% <ø> (ø)
.../org/apache/pinot/client/SimpleBrokerSelector.java	83.33% <0.00%> (-16.67%)	⬇️
.../org/apache/pinot/core/common/MinionConstants.java	0.00% <0.00%> (ø)
...t/core/plan/AggregationGroupByOrderByPlanNode.java	45.00% <0.00%> (ø)
...he/pinot/core/plan/AggregationGroupByPlanNode.java	42.10% <0.00%> (-42.11%)	⬇️
...rg/apache/pinot/core/plan/AggregationPlanNode.java	36.66% <0.00%> (-50.00%)	⬇️
...re/segment/processing/framework/SegmentConfig.java	87.50% <ø> (ø)
...t/core/startree/plan/StarTreeDocIdSetPlanNode.java	0.00% <0.00%> (-100.00%)	⬇️
... and 393 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fe83e95...f7d6bf4. Read the comment docs.