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

    https://github.com/apache/zookeeper/pull/447#discussion_r205332227
  
    --- Diff: 
src/java/main/org/apache/zookeeper/server/quorum/UpgradeableSessionTracker.java 
---
    @@ -19,6 +19,8 @@
     
     import java.util.concurrent.ConcurrentHashMap;
     import java.util.concurrent.ConcurrentMap;
    +import java.util.Set;
    +import java.util.HashSet;
    --- End diff --
    
    Are both imports needed? I don't see any new data structure introduced. 


---

Reply via email to