You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by th...@apache.org on 2016/03/10 19:18:04 UTC

hive git commit: HIVE-13144 : HS2 can leak ZK ACL objects when curator retries to create the persistent ephemeral node (Vaibhav Gumashta via Thejas Nair)

Repository: hive
Updated Branches:
  refs/heads/master 0da77af0e -> 1e8a31e8f


HIVE-13144 : HS2 can leak ZK ACL objects when curator retries to create the persistent ephemeral node (Vaibhav Gumashta via Thejas Nair)


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

Branch: refs/heads/master
Commit: 1e8a31e8fc6aa4ef919b6089e35cc278e66a3b98
Parents: 0da77af
Author: Vaibhav Gumashta <vg...@hortonworks.com>
Authored: Thu Mar 10 10:17:54 2016 -0800
Committer: Thejas Nair <th...@hortonworks.com>
Committed: Thu Mar 10 10:17:54 2016 -0800

----------------------------------------------------------------------
 service/src/java/org/apache/hive/service/server/HiveServer2.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hive/blob/1e8a31e8/service/src/java/org/apache/hive/service/server/HiveServer2.java
----------------------------------------------------------------------
diff --git a/service/src/java/org/apache/hive/service/server/HiveServer2.java b/service/src/java/org/apache/hive/service/server/HiveServer2.java
index 892a476..ab834b9 100644
--- a/service/src/java/org/apache/hive/service/server/HiveServer2.java
+++ b/service/src/java/org/apache/hive/service/server/HiveServer2.java
@@ -220,10 +220,10 @@ public class HiveServer2 extends CompositeService {
    * ACLProvider for providing appropriate ACLs to CuratorFrameworkFactory
    */
   private final ACLProvider zooKeeperAclProvider = new ACLProvider() {
-    List<ACL> nodeAcls = new ArrayList<ACL>();
 
     @Override
     public List<ACL> getDefaultAcl() {
+      List<ACL> nodeAcls = new ArrayList<ACL>();
       if (UserGroupInformation.isSecurityEnabled()) {
         // Read all to the world
         nodeAcls.addAll(Ids.READ_ACL_UNSAFE);