Hello Marton Greber, Alexey Serbin, Gabriella Lotz, Ashwani Raina, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/24287
to look at the new patch set (#5).
Change subject: KUDU-3762 - Fix catchup deadlock on LMP mismatch
......................................................................
KUDU-3762 - Fix catchup deadlock on LMP mismatch
A Raft leader could enter an infinite LMP mismatch loop if it erroneously
marked a peer as unrecoverable (wal_catchup_possible = false). In this
state, RequestForPeer() skips log-position negotiation and sends status
updates at the log tip. If the peer is behind, it rejects the request,
triggering an immediate retry loop that consumes CPU and prevents catchup.
This patch introduces is_joining_existing_cluster_ in CatalogManager to
silence heartbeats and background tasks during the join phase. Explicitly
shuts down the system catalog tablet before initiating AddMaster() to
prevent the leader from seeing a stale Index 0 state.
Testing:
- Added a test case to reproduce the issue.
- Verified the fix resolves the loop and allows the new master to become
a voter.
Change-Id: I00f4f840c21cc037a2aa6c023af168352fef0761
---
M src/kudu/master/catalog_manager.cc
M src/kudu/master/catalog_manager.h
M src/kudu/master/dynamic_multi_master-test.cc
M src/kudu/master/master_runner.cc
M src/kudu/mini-cluster/external_mini_cluster-test.cc
5 files changed, 188 insertions(+), 7 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/87/24287/5
--
To view, visit http://gerrit.cloudera.org:8080/24287
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I00f4f840c21cc037a2aa6c023af168352fef0761
Gerrit-Change-Number: 24287
Gerrit-PatchSet: 5
Gerrit-Owner: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Ashwani Raina <[email protected]>
Gerrit-Reviewer: Gabriella Lotz <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <[email protected]>
