Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3610 to look at the new patch set (#5). Change subject: KUDU-1358 (part 2): heartbeat to every master ...................................................................... KUDU-1358 (part 2): heartbeat to every master Now that followers accept heartbeats, let's modify the tserver to send one to every master. Spawning a heartbeater thread for each master seemed like the natural way to do this; it should simplify dynamic master changes in the future (i.e. just add or remove threads as needed). The "dirty tablet" state is now encapsulated in the heartbeater threads themselves, and the heartbeater must "fan out" to manipulate all of it. It's a little noisy but I think it's reasonable. The alternative is for this state to remain in the TSTabletManager, for the heartbeater to continue tracking which master is the leader, and for it to only send tablet reports to that master. This can be done with a few changes (e.g. adding term numbers to the heartbeat response), but the only benefit is reduced network traffic when tablets are dirty, so that didn't seem worth the complexity. There's no new test here, but this code path is exercised in the test I reenabled, and in the new stress test (follow-on patch). Change-Id: Ic85ac4193462d21c989dbd7874b451e8eaab8e3e --- M src/kudu/integration-tests/master_failover-itest.cc M src/kudu/integration-tests/ts_tablet_manager-itest.cc M src/kudu/tserver/heartbeater.cc M src/kudu/tserver/heartbeater.h M src/kudu/tserver/ts_tablet_manager-test.cc M src/kudu/tserver/ts_tablet_manager.cc M src/kudu/tserver/ts_tablet_manager.h 7 files changed, 316 insertions(+), 267 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/10/3610/5 -- To view, visit http://gerrit.cloudera.org:8080/3610 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ic85ac4193462d21c989dbd7874b451e8eaab8e3e Gerrit-PatchSet: 5 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Dan Burkert <d...@cloudera.com> Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org>