Github user maoling commented on a diff in the pull request:

    https://github.com/apache/zookeeper/pull/490#discussion_r175772100
  
    --- Diff: 
src/java/systest/org/apache/zookeeper/test/system/InstanceManager.java ---
    @@ -266,7 +282,7 @@ synchronized public void removeInstance(String name) 
throws InterruptedException
             if (assigned == null) {
                 return;
             }
    -        assignments.get(assigned.container).remove(name);
    +        assignments.get(assigned.container).remove(new Assigned(name, 0));
             doDelete(assignmentsNode + '/' + assigned.container + '/' + name);
    --- End diff --
    
    +1


---

Reply via email to