morrySnow opened a new pull request, #64787:
URL: https://github.com/apache/doris/pull/64787
### What problem does this PR solve?
Issue Number: N/A
Related PR: N/A
Problem Summary: Aggregate function ORDER BY arguments that contain
expressions requiring pushdown, such as window functions or scalar subqueries,
should push down the real order expression instead of the OrderExpression
wrapper. Otherwise aggregate normalization can place an invalid order wrapper
in the lower project and fail analysis. This change unwraps OrderExpression
only for pushdown collection while preserving the existing behavior for other
aggregate arguments that require pushdown, and adds FE unit and regression
coverage for group_concat ORDER BY window and subquery cases.
### Release note
None
### Check List (For Author)
- Test:
- Unit Test: ./run-fe-ut.sh --run
org.apache.doris.nereids.rules.analysis.NormalizeAggregateTest
- Regression test: ./run-regression-test.sh --run -d query_p0/aggregate
-s agg_group_concat -forceGenOut
- Behavior changed: No
- Does this need documentation: No
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
