Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/466#discussion_r182459083 --- Diff: src/java/main/org/apache/zookeeper/server/command/MonitorCommand.java --- @@ -75,9 +79,9 @@ public void commandRun() { print("synced_followers", leader.getForwardingFollowers().size()); print("pending_syncs", leader.getNumPendingSyncs()); - print("last_proposal_size", leader.getProposalStats().getLastProposalSize()); - print("max_proposal_size", leader.getProposalStats().getMaxProposalSize()); - print("min_proposal_size", leader.getProposalStats().getMinProposalSize()); + print("last_proposal_size", leader.getProposalStats().getLast()); --- End diff -- Agreed, javadoc is misleading. Actually I wanted to keep these two things together and would like to refactor both in a later commit. They refer to the same thing basically, exposing statistics on Jute buffer usage which is a feed of int values. I'm happy to find a better name for the class, but wouldn't create a separate one for almost the same purpose. What do you think of `BufferStats`?
---