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

    https://github.com/apache/zookeeper/pull/622#discussion_r217743312
  
    --- Diff: 
src/java/test/org/apache/zookeeper/server/quorum/FuzzySnapshotRelatedTest.java 
---
    @@ -60,14 +60,17 @@
     
         MainThread[] mt = null;
         ZooKeeper[] zk = null;
    +    int[] clientPorts = null;
         int leaderId;
         int followerA;
     
         @Before
         public void setup() throws Exception {
    +        ZooKeeperServer.setCloseSessionTxnEnabled(true);
    --- End diff --
    
    It would be slightly cleaner to set it back to false in @After method.
    Out of interest, I find it a little bit strange to implement a feature 
switch to a bugfix, but on the other side I understand the concerns.


---

Reply via email to