cloud-fan commented on PR #58942:
URL: https://github.com/apache/spark/pull/58942#issuecomment-5777524577
The as-held-layout fixes are in place, but I found two remaining boundary
issues: the ungrouped waiver currently includes SortMergeAsOfJoinExec even
though EnsureRequirements never builds its spread/replicate pairing, and
non-join cogroup operators lost the old mutual compatibility check for
already-grouped children. Please separate the producer-specific waiver from the
generic multi-child compatibility validation and add focused AS-OF plus
grouped-cogroup coverage.
<!-- SPARK_DEV_REVIEW_REPLY
{"feedback_id":"conversation:5776592901","thread_id":"conversation:5776592901","verdict_sha256":"0d655853a520afc3453e956a11e8a41dba7a05ec2c7ad3cb830af90b05ced131"}
-->
--
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]
