jhungund commented on PR #3435: URL: https://github.com/apache/hive/pull/3435#issuecomment-1202220981
> Yes, I did not know this was an exiting PR that was resurrected. Yes, I thought that keeping table logic where it was was more logical, DeferredWorkHelperImpl doesn't need to know about tables, and doesn't need to be changed in the future if we find more of these issues where we look at an object in different places and get different results. > > Otherwise, seems fine. Thanks Francis. I have tried to address your comment in the latest patchset. Please take a look. Thanks, Janardhan -- 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]
