Zoltan Haindrich created HIVE-22562:
---------------------------------------

             Summary: Harmonize SessionState.getUserName
                 Key: HIVE-22562
                 URL: https://issues.apache.org/jira/browse/HIVE-22562
             Project: Hive
          Issue Type: Improvement
            Reporter: Zoltan Haindrich
            Assignee: Zoltan Haindrich


we might have 2 different user names at the same time:

* 
[getUserName()|https://github.com/apache/hive/blob/ab71e5a22834b5fdd17d6e4ddb54bcd324ae97d7/ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java#L1912]
** a method which relies on the userName field of the SessionState
* 
[getUserFromAuthenticator()|https://github.com/apache/hive/blob/ab71e5a22834b5fdd17d6e4ddb54bcd324ae97d7/ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java#L1291]
** a method which uses the authenticator to do the heavy lifting
* there all kind of interesting call sites like:
** there are some which are [prefering the authenticator over 
getUserName()|https://github.com/apache/hive/blob/ab71e5a22834b5fdd17d6e4ddb54bcd324ae97d7/ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionPoolManager.java#L254]
** there are some which [use getUserName() regardless authenticator, but have 
fixme|https://github.com/apache/hive/blob/ab71e5a22834b5fdd17d6e4ddb54bcd324ae97d7/ql/src/java/org/apache/hadoop/hive/ql/Driver.java#L1669]
** and there are some which are just using the authenticator with or without 
notes/etc





--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to