szehon-ho commented on PR #58412: URL: https://github.com/apache/spark/pull/58412#issuecomment-5501803520
Thanks for the detailed review. The merge-order concern is now resolved: #58439 landed in both master (6864fbbbc3f) and branch-4.x (91c3e7d5ea1). Both commits have the same patch ID as c845783a906 in this PR. The transform-semantics changes are limited to test fixtures. I propose handling their upstream synchronization separately rather than expanding this 4.3 backport. Items 3–8 likewise describe existing upstream behavior or cleanup and should be addressed upstream first if pursued. -- 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]
