You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by jc...@apache.org on 2018/08/04 00:49:43 UTC
hive git commit: Revert "HIVE-20118:
SessionStateUserAuthenticator.getGroupNames() is always empty (Daniel Dai,
reviewed by Thejas Nair)"
Repository: hive
Updated Branches:
refs/heads/master b02012f62 -> ce2754daf
Revert "HIVE-20118: SessionStateUserAuthenticator.getGroupNames() is always empty (Daniel Dai, reviewed by Thejas Nair)"
This reverts commit b02012f6282094b2af2d66ec1973da6082fa48d8.
Project: http://git-wip-us.apache.org/repos/asf/hive/repo
Commit: http://git-wip-us.apache.org/repos/asf/hive/commit/ce2754da
Tree: http://git-wip-us.apache.org/repos/asf/hive/tree/ce2754da
Diff: http://git-wip-us.apache.org/repos/asf/hive/diff/ce2754da
Branch: refs/heads/master
Commit: ce2754daf0e33fd9787eaeaeaa0472c73637ad97
Parents: b02012f
Author: Jesus Camacho Rodriguez <jc...@apache.org>
Authored: Fri Aug 3 17:49:34 2018 -0700
Committer: Jesus Camacho Rodriguez <jc...@apache.org>
Committed: Fri Aug 3 17:49:34 2018 -0700
----------------------------------------------------------------------
.../hive/ql/security/SessionStateUserAuthenticator.java | 7 +------
1 file changed, 1 insertion(+), 6 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hive/blob/ce2754da/ql/src/java/org/apache/hadoop/hive/ql/security/SessionStateUserAuthenticator.java
----------------------------------------------------------------------
diff --git a/ql/src/java/org/apache/hadoop/hive/ql/security/SessionStateUserAuthenticator.java b/ql/src/java/org/apache/hadoop/hive/ql/security/SessionStateUserAuthenticator.java
index 1ded295..3bf990a 100644
--- a/ql/src/java/org/apache/hadoop/hive/ql/security/SessionStateUserAuthenticator.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/security/SessionStateUserAuthenticator.java
@@ -24,7 +24,6 @@ import java.util.List;
import org.apache.hadoop.conf.Configuration;
import org.apache.hadoop.hive.ql.metadata.HiveException;
import org.apache.hadoop.hive.ql.session.SessionState;
-import org.apache.hadoop.security.UserGroupInformation;
/**
* Authenticator that returns the userName set in SessionState. For use when authorizing with HS2
@@ -36,14 +35,10 @@ public class SessionStateUserAuthenticator implements HiveAuthenticationProvider
protected Configuration conf;
private SessionState sessionState;
- private List<String> groups;
@Override
public List<String> getGroupNames() {
- if (groups == null) {
- groups = UserGroupInformation.createRemoteUser(sessionState.getUserName()).getGroups();
- }
- return groups;
+ return groupNames;
}
@Override