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)