srowen commented on pull request #29370: URL: https://github.com/apache/spark/pull/29370#issuecomment-670006325
This change itself looks fine. If you think the rest of the changes in sql / catalyst are similar, I'd make them in one go. If there are difficult or logically distinct other fixes, that can be another PR. They can all be attached to one JIRA ---------------------------------------------------------------- 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: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org