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 <ser...@apache.org> Authored: Fri May 6 12:27:03 2016 -0700 Committer: Sergey Shelukhin <ser...@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);