vanzin commented on issue #26416: [SPARK-29779][CORE] Compact old event log files and cleanup URL: https://github.com/apache/spark/pull/26416#issuecomment-570643139 Yes, reviewers still need the big picture, but smaller reviews are easier to do and keep the code focused. They also help you with designing proper interfaces. There are different ways to give context to others. e.g. #27053 which keeps the full body work in a separate PR for reference. In the past I've kept separate, incremental PRs on my own clone and linked from my Spark PRs, but that's a lot more work.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
