Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3501 to look at the new patch set (#5). Change subject: KUDU-763 consensus queue metrics on followers are messed up ...................................................................... KUDU-763 consensus queue metrics on followers are messed up On non-leader tablet replicas, the majority_done_ops and in_progress_ops metrics are wrong. The reason is that non-leaders do not update their consensus queue's committed index (queue_state_.committed_index). This patch hides these incorrect numbers in the web ui, where they were previously exposed (I think unintentionally, the code having printed the queue when it meant to print the state), documents that the numbers are not meaningful except in the leader case, and forces them to always be zero whenever the tablet replica is not a leader. Change-Id: I9fb0d45f85786b9e2631b5dc0bf044a9d3192a39 --- M src/kudu/consensus/consensus_queue-test.cc M src/kudu/consensus/consensus_queue.cc M src/kudu/consensus/consensus_queue.h M src/kudu/consensus/raft_consensus.cc 4 files changed, 68 insertions(+), 10 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/01/3501/5 -- To view, visit http://gerrit.cloudera.org:8080/3501 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I9fb0d45f85786b9e2631b5dc0bf044a9d3192a39 Gerrit-PatchSet: 5 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Will Berkeley <wdberke...@gmail.com> Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com>