dlmarion commented on code in PR #5256: URL: https://github.com/apache/accumulo/pull/5256#discussion_r1928891372
########## minicluster/src/main/java/org/apache/accumulo/miniclusterImpl/MiniAccumuloClusterImpl.java: ########## @@ -1003,31 +1002,23 @@ public synchronized void stop() throws IOException, InterruptedException { } catch (RuntimeException e) { log.error("Error zapping zookeeper locks", e); } - control.stop(ServerType.ZOOKEEPER, null); // Clear the location of the servers in ZooCache. - // When ZooKeeper was stopped in the previous method call, - // the local ZooKeeper watcher did not fire. If MAC is - // restarted, then ZooKeeper will start on the same port with - // the same data, but no Watchers will fire. - boolean startCalled = true; + // If MAC is restarted, then ZooKeeper will start + // on the same port with the same data, but no Watchers will fire. Review Comment: removed this comment in a481f9b -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@accumulo.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org