HyukjinKwon edited a comment on pull request #31541: URL: https://github.com/apache/spark/pull/31541#issuecomment-776940787
@rdblue, please read my comments above and the previous PR. Your point that I simply misunderstood is incorrect. My reason to revert this change is that we don't allow that in the default Spark session as I explained above and the previous PR. It should at least to be discussed before committing related changes. At the very least it has to be explained in the PR. Was this explained? Also do you mean we're allowing to overwrite the default Spark session by design, @rdblue? If that's the case we should revisit all changes made for the Spark default session, and multi parts validation. This is also not about policy but more about the best practice. Do you mean it is healthy to commit changes without getting reviewed from original author, reviewer or other committers? ---------------------------------------------------------------- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
