You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by sp...@apache.org on 2015/10/19 01:40:47 UTC

hive git commit: HIVE-12188: DoAs does not work properly in non-kerberos secured HS2 (Chaoyu Tang, reviewed by Sergio Pena)

Repository: hive
Updated Branches:
  refs/heads/master 1969f8ab7 -> 1450c09c3


HIVE-12188: DoAs does not work properly in non-kerberos secured HS2 (Chaoyu Tang, reviewed by Sergio Pena)


Project: http://git-wip-us.apache.org/repos/asf/hive/repo
Commit: http://git-wip-us.apache.org/repos/asf/hive/commit/1450c09c
Tree: http://git-wip-us.apache.org/repos/asf/hive/tree/1450c09c
Diff: http://git-wip-us.apache.org/repos/asf/hive/diff/1450c09c

Branch: refs/heads/master
Commit: 1450c09c308a491c49407c415ed0882a03439389
Parents: 1969f8a
Author: Sergio Pena <se...@cloudera.com>
Authored: Sun Oct 18 18:39:21 2015 -0500
Committer: Sergio Pena <se...@cloudera.com>
Committed: Sun Oct 18 18:39:21 2015 -0500

----------------------------------------------------------------------
 .../org/apache/hive/service/cli/thrift/ThriftCLIService.java    | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hive/blob/1450c09c/service/src/java/org/apache/hive/service/cli/thrift/ThriftCLIService.java
----------------------------------------------------------------------
diff --git a/service/src/java/org/apache/hive/service/cli/thrift/ThriftCLIService.java b/service/src/java/org/apache/hive/service/cli/thrift/ThriftCLIService.java
index 1c3e899..0532d79 100644
--- a/service/src/java/org/apache/hive/service/cli/thrift/ThriftCLIService.java
+++ b/service/src/java/org/apache/hive/service/cli/thrift/ThriftCLIService.java
@@ -434,14 +434,11 @@ public abstract class ThriftCLIService extends AbstractService implements TCLISe
 
   private String getDelegationToken(String userName)
       throws HiveSQLException, LoginException, IOException {
-    if (userName == null || !cliService.getHiveConf().getVar(ConfVars.HIVE_SERVER2_AUTHENTICATION)
-        .equalsIgnoreCase(HiveAuthFactory.AuthTypes.KERBEROS.toString())) {
-      return null;
-    }
     try {
       return cliService.getDelegationTokenFromMetaStore(userName);
     } catch (UnsupportedOperationException e) {
       // The delegation token is not applicable in the given deployment mode
+      // such as HMS is not kerberos secured 
     }
     return null;
   }