Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/10076 )
Change subject: KUDU-2287 Expose election failures as metrics ...................................................................... Patch Set 23: (4 comments) http://gerrit.cloudera.org:8080/#/c/10076/22/src/kudu/consensus/raft_consensus.h File src/kudu/consensus/raft_consensus.h: http://gerrit.cloudera.org:8080/#/c/10076/22/src/kudu/consensus/raft_consensus.h@871 PS22, Line 871: num_failed_elections_met > nit: rename to num_failed_elections_metric_ Done http://gerrit.cloudera.org:8080/#/c/10076/22/src/kudu/consensus/raft_consensus.cc File src/kudu/consensus/raft_consensus.cc: http://gerrit.cloudera.org:8080/#/c/10076/22/src/kudu/consensus/raft_consensus.cc@1293 PS22, Line 1293: // The deduplicated request. > This should go down where we snooze the failure detector below line 1316, i Done http://gerrit.cloudera.org:8080/#/c/10076/22/src/kudu/consensus/raft_consensus.cc@2552 PS22, Line 2552: // This could be a problem during tablet deletion. > Put this inside BecomeLeaderUnlocked() Done http://gerrit.cloudera.org:8080/#/c/10076/22/src/kudu/consensus/raft_consensus.cc@3000 PS22, Line 3000: > nit: rename this to GetMillisSinceLastLeaderHeartbeat() Done -- To view, visit http://gerrit.cloudera.org:8080/10076 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I1b25df258cdba7bdae7bb2d7b4eb3d73b53425c3 Gerrit-Change-Number: 10076 Gerrit-PatchSet: 23 Gerrit-Owner: Attila Bukor <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Reviewer: Will Berkeley <[email protected]> Gerrit-Comment-Date: Fri, 01 Jun 2018 04:27:36 +0000 Gerrit-HasComments: Yes
