spark icon indicating copy to clipboard operation
spark copied to clipboard
verified publisher icon apache

[SPARK-48206][SQL][TESTS] Add tests for window rewrites with RewriteWithExpression

Open kelvinjian-db opened this issue 1 year ago • 3 comments

What changes were proposed in this pull request?

This PR adds more testing for RewriteWithExpression around Window operators.

Why are the changes needed?

Adds more testing for RewriteWithExpression, which can be fragile around WindowExpressions.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Unit tests.

Was this patch authored or co-authored using generative AI tooling?

No.

kelvinjian-db avatar May 09 '24 01:05 kelvinjian-db

@cloud-fan

kelvinjian-db avatar May 09 '24 18:05 kelvinjian-db

sorry it has conflicts now

cloud-fan avatar May 10 '24 17:05 cloud-fan

fixed

kelvinjian-db avatar May 10 '24 20:05 kelvinjian-db

thanks, merging to master!

cloud-fan avatar May 13 '24 14:05 cloud-fan