Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14976 )
Change subject: KUDU-3011 p1: metric to count tablet leaders ...................................................................... KUDU-3011 p1: metric to count tablet leaders Adds a metric that tracks the number of tablet leaders on a given tablet server. This is done by plumbing down a top-level gauge down from the KuduServer to each TabletReplica's RaftConsensus instance. This is done at the KuduServer to eventually be extensible to count the number of master system catalog leaders, though that plumbing is left for a future patch. I've left a TODO where I expect this to happen. I also considered instead having the metric be defined by a functor that would iterate through all replicas and check each's leadership status. I opted to not do this, since iterating through and locking each RaftConsensus instance seemed like it'd be less performant. This will be useful in orchestrating a smooth maintenance window, as it will allow us to determine whether leadership has quiesced away from a given tablet server. Change-Id: Iaa6554458a860e34f97af168da7ed786c8ef47e4 Reviewed-on: http://gerrit.cloudera.org:8080/14976 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> --- M src/kudu/consensus/raft_consensus.cc M src/kudu/consensus/raft_consensus.h M src/kudu/consensus/raft_consensus_quorum-test.cc M src/kudu/integration-tests/raft_consensus_election-itest.cc M src/kudu/integration-tests/ts_tablet_manager-itest.cc M src/kudu/kserver/kserver.cc M src/kudu/kserver/kserver.h M src/kudu/master/sys_catalog.cc M src/kudu/tablet/tablet_replica-test.cc M src/kudu/tablet/tablet_replica.cc M src/kudu/tablet/tablet_replica.h M src/kudu/tserver/tablet_copy_source_session-test.cc M src/kudu/tserver/ts_tablet_manager.cc 13 files changed, 163 insertions(+), 36 deletions(-) Approvals: Kudu Jenkins: Verified Adar Dembo: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/14976 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Iaa6554458a860e34f97af168da7ed786c8ef47e4 Gerrit-Change-Number: 14976 Gerrit-PatchSet: 9 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241)
