jcamachor commented on pull request #1147: URL: https://github.com/apache/hive/pull/1147#issuecomment-668281996
@maheshk114 , thanks for addressing the first batch of comments. PR looks better. I have done a second pass and left some additional comments that should be addressed before merging. Please, also merge master into your branch, since there seem to be some conflicts. ---------------------------------------------------------------- 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: gitbox-unsubscr...@hive.apache.org For additional commands, e-mail: gitbox-h...@hive.apache.org