[jira] [Comment Edited] (HIVE-22562) Harmonize SessionState.getUserName

["Peter Vary (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/HIVE-22562?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16984977#comment-16984977 ] 

Peter Vary edited comment on HIVE-22562 at 11/29/19 12:39 PM:
--------------------------------------------------------------

[~kgyrtkirk]: When I was playing around with current_user and created logged_in_user UDF (HIVE-14100), I have found in some configurations only the current_user provided meaningful information, in other configurations only the logged_in_user. I am not sure, but I think it had to do something with the impersonation related settings. So please consider this when moving forward. (Sorry that I am not able to remember the exact details :()
 Thanks,
 Peter


was (Author: pvary):
[~kgyrtkirk]: When I was playing around with current_user and created logged_in_user UDF, I have found in some configurations only the current_user provided meaningful information, in other configurations only the logged_in_user. I am not sure, but I think it had to do something with the impersonation related settings. So please consider this when moving forward. (Sorry that I am not able to remember the exact details :()
 Thanks,
 Peter

> 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
>            Priority: Major
>
> 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)