dongjoon-hyun commented on PR #55538: URL: https://github.com/apache/spark/pull/55538#issuecomment-4314730331
I got your point, but it would be great if we can have a small unit test case specifically to prevent any regressions (not from this PR) for future proof. > No new tests needed -- the fix aligns the implementation with the contract that EnsureRequirements already relies on. Existing KeyGroupedPartitioningSuite and related suites cover the behaviour. -- 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]
