You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by se...@apache.org on 2016/05/06 19:27:37 UTC

hive git commit: HIVE-13710 : LLAP registry ACL check causes error due to namespacing (Sergey Shelukhin, reviewed by Prasanth Jayachandran)

Repository: hive
Updated Branches:
  refs/heads/master f089f2e64 -> 92a9ae800


HIVE-13710 : LLAP registry ACL check causes error due to namespacing (Sergey Shelukhin, reviewed by Prasanth Jayachandran)


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

Branch: refs/heads/master
Commit: 92a9ae80003beb2f94d94f7ed0a1cdf168534930
Parents: f089f2e
Author: Sergey Shelukhin <se...@apache.org>
Authored: Fri May 6 12:27:03 2016 -0700
Committer: Sergey Shelukhin <se...@apache.org>
Committed: Fri May 6 12:27:28 2016 -0700

----------------------------------------------------------------------
 .../hadoop/hive/llap/registry/impl/LlapZookeeperRegistryImpl.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hive/blob/92a9ae80/llap-client/src/java/org/apache/hadoop/hive/llap/registry/impl/LlapZookeeperRegistryImpl.java
----------------------------------------------------------------------
diff --git a/llap-client/src/java/org/apache/hadoop/hive/llap/registry/impl/LlapZookeeperRegistryImpl.java b/llap-client/src/java/org/apache/hadoop/hive/llap/registry/impl/LlapZookeeperRegistryImpl.java
index fde70e7..cffa493 100644
--- a/llap-client/src/java/org/apache/hadoop/hive/llap/registry/impl/LlapZookeeperRegistryImpl.java
+++ b/llap-client/src/java/org/apache/hadoop/hive/llap/registry/impl/LlapZookeeperRegistryImpl.java
@@ -328,7 +328,7 @@ public class LlapZookeeperRegistryImpl implements ServiceRegistry {
     if (ix > 0) {
       pathToCheck = pathToCheck.substring(0, ix);
     }
-    List<ACL> acls = zooKeeperClient.usingNamespace(null).getACL().forPath(pathToCheck);
+    List<ACL> acls = zooKeeperClient.getACL().forPath(pathToCheck);
     if (acls == null || acls.isEmpty()) {
       // Can there be no ACLs? There's some access (to get ACLs), so assume it means free for all.
       throw new SecurityException("No ACLs on "  + pathToCheck);