morrySnow opened a new pull request, #65537:
URL: https://github.com/apache/doris/pull/65537
### What problem does this PR solve?
Issue Number: None
Related PR: None
Problem Summary: NormalizeAggregate only checked the first child of each
aggregate function for nested aggregate functions. Aggregate functions in later
arguments, such as SUM in the ORDER BY argument of GROUP_CONCAT, could bypass
analysis validation. Check every aggregate child and add FE unit and regression
coverage for the invalid query.
### Release note
Reject nested aggregate functions in every aggregate function argument.
### Check List (For Author)
- Test: Unit Test and Regression test
- Unit Test: NormalizeAggregateTest
- Regression test: query_p0/aggregate/agg_group_concat
- Behavior changed: Yes, invalid nested aggregate functions in non-first
arguments are rejected during analysis
- 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]
