[bugfix] Agg literal attach should happen after BASIC_RULES

[walterddr]

this fixes #11862

[Jackie-Jiang]

Please add a test for this

[codecov-commenter]

Codecov Report

Merging #11863 (871d6dd) into master (0174c71) will decrease coverage by 0.17%.
Report is 44 commits behind head on master.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master   #11863      +/-   ##
============================================
- Coverage     62.99%   62.83%   -0.17%     
  Complexity     1140     1140              
============================================
  Files          2343     2367      +24     
  Lines        126305   127943    +1638     
  Branches      19419    19744     +325     
============================================
+ Hits          79563    80388     +825     
- Misses        41071    41826     +755     
- Partials       5671     5729      +58     

Flag	Coverage Δ
custom-integration1	<0.01% <0.00%> (?)
integration	<0.01% <0.00%> (-0.01%)	⬇️
integration1	<0.01% <0.00%> (-0.01%)	⬇️
integration2	0.00% <0.00%> (ø)
java-11	14.42% <0.00%> (+0.01%)	⬆️
java-17	?
java-20	?
java-21	62.82% <100.00%> (?)
skip-bytebuffers-false	62.79% <100.00%> (?)
skip-bytebuffers-true	62.80% <100.00%> (?)
temurin	62.83% <100.00%> (-0.17%)	⬇️
unittests	62.82% <100.00%> (-0.17%)	⬇️
unittests1	66.83% <100.00%> (-0.27%)	⬇️
unittests2	14.42% <0.00%> (+0.01%)	⬆️

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

Files	Coverage Δ
.../java/org/apache/pinot/query/QueryEnvironment.java	90.59% <100.00%> (ø)

... and 149 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more