Pass literal within AggregateCall via rexList

[Jackie-Jiang]

Another try of #13217

• In PinotAggregateExchangeNodeInsertRule, reconstruct all AggregateCall to carry the literal within the newly created AggregateCall
• Address the arguments logic for aggregation within PinotAggregateExchangeNodeInsertRule to significantly simplify the special handling in AggregateOperator
• Fix the bug of not allowing using LEAF to process distinct aggregates
• Fix the bug of wrong plan potentially created when collation exists

[codecov-commenter]

Codecov Report

❌ Patch coverage is 89.30818% with 17 lines in your changes missing coverage. Please review.
✅ Project coverage is 62.16%. Comparing base (59551e4) to head (107874c).
⚠️ Report is 2926 commits behind head on master.

Files with missing lines	Patch %	Lines
...el/rules/PinotAggregateExchangeNodeInsertRule.java	89.62%	9 Missing and 5 partials ⚠️
...inot/query/runtime/operator/AggregateOperator.java	78.57%	1 Missing and 2 partials ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #13282      +/-   ##
============================================
+ Coverage     61.75%   62.16%   +0.40%     
+ Complexity      207      198       -9     
============================================
  Files          2436     2534      +98     
  Lines        133233   139300    +6067     
  Branches      20636    21530     +894     
============================================
+ Hits          82274    86590    +4316     
- Misses        44911    46234    +1323     
- Partials       6048     6476     +428     

Flag	Coverage Δ
custom-integration1	<0.01% <0.00%> (-0.01%)	⬇️
integration	<0.01% <0.00%> (-0.01%)	⬇️
integration1	<0.01% <0.00%> (-0.01%)	⬇️
integration2	0.00% <0.00%> (ø)
java-11	62.11% <89.30%> (+0.40%)	⬆️
java-21	62.03% <89.30%> (+0.40%)	⬆️
skip-bytebuffers-false	62.13% <89.30%> (+0.38%)	⬆️
skip-bytebuffers-true	62.01% <89.30%> (+34.28%)	⬆️
temurin	62.16% <89.30%> (+0.40%)	⬆️
unittests	62.15% <89.30%> (+0.40%)	⬆️
unittests1	46.72% <89.30%> (-0.17%)	⬇️
unittests2	27.76% <0.00%> (+0.02%)	⬆️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[xiangfu0]

will this be a function call e.g. array of literals?

[Jackie-Jiang]

It won't be. We only extract literal as operand, others will remain input reference.