Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/14976 )
Change subject: KUDU-3011 p1: metric to count tablet leaders ...................................................................... Patch Set 5: (2 comments) http://gerrit.cloudera.org:8080/#/c/14976/4/src/kudu/consensus/raft_consensus.cc File src/kudu/consensus/raft_consensus.cc: http://gerrit.cloudera.org:8080/#/c/14976/4/src/kudu/consensus/raft_consensus.cc@699 PS4, Line 699: > It's not possible to do a LEADER->LEADER transition and thus double count o I couldn't think of such a scenario, but I'll add some logic for it just in case. http://gerrit.cloudera.org:8080/#/c/14976/3/src/kudu/master/sys_catalog.cc File src/kudu/master/sys_catalog.cc: http://gerrit.cloudera.org:8080/#/c/14976/3/src/kudu/master/sys_catalog.cc@382 PS3, Line 382: // TODO(awong): plumb master_->num_raft_leaders() here. > I'd be open to moving it to kserver. But a follow-up patch is fine if that' I moved it into kserver, but I'll leave the plumbing and testing for master to a different patch. -- 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: comment Gerrit-Change-Id: Iaa6554458a860e34f97af168da7ed786c8ef47e4 Gerrit-Change-Number: 14976 Gerrit-PatchSet: 5 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) Gerrit-Comment-Date: Mon, 06 Jan 2020 06:43:57 +0000 Gerrit-HasComments: Yes
