veghlaci05 commented on code in PR #3402:
URL: https://github.com/apache/hive/pull/3402#discussion_r905882417
##########
ql/src/java/org/apache/hadoop/hive/ql/exec/tez/YarnQueueHelper.java:
##########
@@ -92,6 +94,20 @@ public YarnQueueHelper(HiveConf conf) {
}
public void checkQueueAccess(
+ String queueName, String userName) throws IOException,
InterruptedException {
+ UserGroupInformation ugi = UserGroupInformation.getCurrentUser();
Review Comment:
I had a discussion regarding this with @abstractdog and he is thinking that
if the current user is changed from the login user somewhere upper in the call
stack, thank we should respect that, since that change had some reason.
--
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]
