You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@zookeeper.apache.org by ma...@apache.org on 2021/01/24 09:26:40 UTC

[zookeeper] branch master updated: ZOOKEEPER-3904: Remove duplicate check in ContainerManager#getCandidates method

This is an automated email from the ASF dual-hosted git repository.

maoling pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/zookeeper.git


The following commit(s) were added to refs/heads/master by this push:
     new 7af6428  ZOOKEEPER-3904: Remove duplicate check in ContainerManager#getCandidates method
7af6428 is described below

commit 7af6428fa825ae5bd3938f3a8fd21a92abd4aadb
Author: Technoboy- <te...@yeah.net>
AuthorDate: Sun Jan 24 17:26:29 2021 +0800

    ZOOKEEPER-3904: Remove duplicate check in ContainerManager#getCandidates method
    
    Author: Technoboy- <te...@yeah.net>
    
    Reviewers: Reviewers: maoling <ma...@sina.com>
    
    Closes #1419 from Technoboy-/dev2
---
 .../src/main/java/org/apache/zookeeper/server/ContainerManager.java    | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/zookeeper-server/src/main/java/org/apache/zookeeper/server/ContainerManager.java b/zookeeper-server/src/main/java/org/apache/zookeeper/server/ContainerManager.java
index 9ecb8c6..7abac58 100644
--- a/zookeeper-server/src/main/java/org/apache/zookeeper/server/ContainerManager.java
+++ b/zookeeper-server/src/main/java/org/apache/zookeeper/server/ContainerManager.java
@@ -181,9 +181,6 @@ public class ContainerManager {
                     }
                 }
             }
-            if ((node != null) && (node.stat.getCversion() > 0) && (node.getChildren().isEmpty())) {
-                candidates.add(containerPath);
-            }
         }
         for (String ttlPath : zkDb.getDataTree().getTtls()) {
             DataNode node = zkDb.getDataTree().getNode(ttlPath);