thomasrebele commented on PR #6276: URL: https://github.com/apache/hive/pull/6276#issuecomment-4178195817
As discussed with Krisztian, it would be nice if the class StatObjectConverter could be refactored: instead of a bunch of if conditions in the class, there would be a factory that creates a converter to do the conversion. The property would then affect the factory to create a converter that uses the longStats field. As this could be a bigger refactoring, it would be better to start it from the latest master branch, and then implement HIVE-29398 on top of it. -- 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]
