Adar Dembo has posted comments on this change.

Change subject: consensus_peers: schedule raft heartbeats through messenger
......................................................................


Patch Set 2:

> Any chance you've run this on a small cluster (even a localhost
 > one) with lots of tablets and three nodes? would be curious to look
 > at a tracing.html output to make sure the heartbeats are not
 > somehow ending up in lockstep, etc. Would also be good to confirm
 > that kill -STOP for a second or two followed by kill -CONT doesn't
 > end up producing lockstep heartbeats, etc.
 > 
 > LMK if you want me to try this locally.

I haven't had a chance to do that yet, and I would appreciate if you could give 
it a shot, since I'm going to be away for a few weeks after today.

-- 
To view, visit http://gerrit.cloudera.org:8080/7331
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iac8e09fe02dd32885ef0cf644cb093b1c8e6afb8
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: No

Reply via email to