Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/580#discussion_r207891221 --- Diff: src/java/main/org/apache/zookeeper/server/ServerStats.java --- @@ -33,17 +34,17 @@ public class ServerStats { private static final Logger LOG = LoggerFactory.getLogger(ServerStats.class); - private long packetsSent; - private long packetsReceived; - private long maxLatency; - private long minLatency = Long.MAX_VALUE; - private long totalLatency = 0; - private long count = 0; + private final AtomicLong packetsSent = new AtomicLong(); --- End diff -- Why are these 3 not part of `ServerMetrics`? Is that a future step of the migration?
---