Yicong-Huang commented on PR #58198: URL: https://github.com/apache/spark/pull/58198#issuecomment-5414768364
> I would have expected a bit more time for people to review this since it's a significant refactoring. In any case, I will try out the dry run on #58214 (after I resolve the merge conflict there)! Fair point, sorry for rushing this. I tested the dry-run mode locally and also merged this PR itself using the updated merge script without dry run. So functionality wise the refactor should work the same as before. Please do let me know if you found any issues! I can quickly address them in follow-up PRs. -- 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]
