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);