thomasrebele commented on PR #6293: URL: https://github.com/apache/hive/pull/6293#issuecomment-4040431572
Thank you for the review and the refactoring, @zabetak! At a first glance the refactoring looks reasonable, and I'll incorporate most of it. I would still keep the `makeRange` function with the additional check just to be sure that no exception will be raised when creating ranges. -- 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]
