Mingliang Zhu
Mingliang Zhu
> @zml1206 The PR description is not clear enough. Please modify `We can extract the limit value 2 (("a", 4, ""),("a", 4, "")) on window group a because window expression...
I changed the description. Do you think it’s easier to understand this way? @cloud-fan
Is there still any problem with this PR, Can it be merged? Can you help take a look @cloud-fan
Thank you all for review. @cloud-fan @kelvinjian-db @beliefer
cc @scottsand-db Do you have time to take a look? Thank you.
Resolved by #3059
Thank you. @cloud-fan @peter-toth
> Can we solve this issue by recursion? Which is easier to understand. `rewriteWithExprAndInputPlans` was originally by recursion, so I optimized the code and comments for easy to understand, please...
CI failed unrelated.
> Is it possible for you to share some test results about this change? Thanks! I ran it in our grayscale adhoc query environment yesterday, and executorCpuTime overall reduced by...