Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Adar Dembo,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/14976
to look at the new patch set (#7).
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 TSTabletManager to each TabletReplica's RaftConsensus instance.
I 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
---
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, 166 insertions(+), 36 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/76/14976/7
--
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: newpatchset
Gerrit-Change-Id: Iaa6554458a860e34f97af168da7ed786c8ef47e4
Gerrit-Change-Number: 14976
Gerrit-PatchSet: 7
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)
