[BugFix] Pushdown distinct agg across window not support complex expr

[satanson]

Why I'm doing:

Pushdown DistinctAgg across window operator does not support complex expr that uses output column of the window operator, it report error when encounters with it, the query report an error as follows:

with temp as ( 
select TIME, ceil(round(sum(`NUM`) OVER (PARTITION BY `TIME` ))) from test_data
) select distinct * from temp;

The error message as follows:

java.lang.IllegalArgumentException: null
	at com.google.common.base.Preconditions.checkArgument(Preconditions.java:129) ~[spark-dpp-1.0.0.jar:?]
	at com.starrocks.sql.optimizer.rule.tree.pdagg.PushDownDistinctAggregateRewriter$PostVisitor.visitLogicalTableScan(PushDownDistinctAggregateRewriter.java:351) ~[starrocks-fe.jar:?]
	at com.starrocks.sql.optimizer.rule.tree.pdagg.PushDownDistinctAggregateRewriter$PostVisitor.visitLogicalTableScan(PushDownDistinctAggregateRewriter.java:311) ~[starrocks-fe.jar:?]
	at com.starrocks.sql.optimizer.operator.logical.LogicalScanOperator.accept(LogicalScanOperator.java:187) ~[starrocks-fe.jar:?]
	at com.starrocks.sql.optimizer.rule.tree.pdagg.PushDownDistinctAggregateRewriter.processPost(PushDownDistinctAggregateRewriter.java:475) ~[starrocks-fe.jar:?]
	at com.starrocks.sql.optimizer.rule.tree.pdagg.PushDownDistinctAggregateRewriter.lambda$process$3(PushDownDistinctAggregateRewriter.java:484) ~[starrocks-fe.jar:?]
	at java.util.Optional.map(Optional.java:215) ~[?:1.8.0_181]
	at com.starrocks.sql.optimizer.rule.tree.pdagg.PushDownDistinctAggregateRewriter.process(PushDownDistinctAggregateRewriter.java:484) ~[starrocks-fe.jar:?]
	at com.starrocks.sql.optimizer.rule.tree.pdagg.PushDownDistinctAggregateRewriter.lambda$processChildren$0(PushDownDistinctAggregateRewriter.java:445) ~[starrocks-fe.jar:?]
	at java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:193) ~[?:1.8.0_181]
	at java.util.ArrayList$ArrayListSpliterator.forEachRemaining(ArrayList.java:1382) ~[?:1.8.0_181]
	at java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:481) ~[?:1.8.0_181]
	at java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:471) ~[?:1.8.0_181]

What I'm doing:

Support the scenarios that disitinct columns are complex exprs that use output columns of window operator.

Fixes #issue

What type of PR is this:

• BugFix
• Feature
• Enhancement
• Refactor
• UT
• Doc
• Tool

Does this PR entail a change in behavior?

• Yes, this PR will result in a change in behavior.
• No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

• Interface/UI changes: syntax, type conversion, expression evaluation, display information
• Parameter changes: default values, similar parameters but with different default values
• Policy changes: use new policy to replace old one, functionality automatically enabled
• Feature removed
• Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

• I have added test cases for my bug fix or my new feature
• This pr needs user documentation (for new or modified features or behaviors)
  • I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

• I have checked the version labels which the pr will be auto-backported to the target branch
  • 3.2
  • 3.1
  • 3.0
  • 2.5

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

0.0% Coverage
0.0% Duplication

The version of Java (11.0.21) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17.
Read more here

[github-actions]

[BE Incremental Coverage Report]

✅ pass : 0 / 0 (0%)

[github-actions]

[FE Incremental Coverage Report]

❌ fail : 4 / 17 (23.53%)

file detail

	path	covered_line	new_line	coverage	not_covered_line_detail
🔵	com/starrocks/sql/optimizer/LogicalPlanPrinter.java	0	13	00.00%	[257, 259, 260, 261, 262, 263, 264, 265, 266, 267, 268, 269, 272]
🔵	com/starrocks/sql/optimizer/rule/tree/pdagg/PushDownDistinctAggregateRewriter.java	4	4	100.00%	[]

[github-actions]

@Mergifyio backport branch-3.2

[github-actions]

@Mergifyio backport branch-3.1

[github-actions]

@Mergifyio backport branch-3.0

[mergify]

backport branch-3.2

✅ Backports have been created

• #36443 [BugFix] Pushdown distinct agg across window not support complex expr (backport #36357) has been created for branch branch-3.2 but encountered conflicts

[mergify]

backport branch-3.1

✅ Backports have been created

• #36444 [BugFix] Pushdown distinct agg across window not support complex expr (backport #36357) has been created for branch branch-3.1 but encountered conflicts

[github-actions]

@Mergifyio backport branch-2.5

[mergify]

backport branch-3.0

✅ Backports have been created

• #36445 [BugFix] Pushdown distinct agg across window not support complex expr (backport #36357) has been created for branch branch-3.0 but encountered conflicts

[mergify]

backport branch-2.5

✅ Backports have been created

• #36446 [BugFix] Pushdown distinct agg across window not support complex expr (backport #36357) has been created for branch branch-2.5 but encountered conflicts