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.
---