yigress commented on PR #4068: URL: https://github.com/apache/hive/pull/4068#issuecomment-1439012689
Thank you @zabetak for pointing to the correct HIVE-21305 that changed permissions for the test files. HIVE-21305 suggests that with IO cache and without IO cache the file permissions will be different, so that gives the impression that in LLAP the permissions is not really an issue? I tested this with non-LLAP hive, there is no permissions change before or after HIVE-27016. As HIVE-27016 invokes Outputcommitter on Tez-AM, so my guess is that in LLAP it causes file write (in these orc file tests) with the changed permissions, however in non-LLAP this only affects storagehandler type tables so the output permissions for orc tables remain unchanged. This calls for difference of outputs file permissions in LLAP and non-LLAP, wonder has this been discussed before? -- 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]
