Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/14946
to look at the new patch set (#2).
Change subject: [consensus] KUDU-2839 fix assert in GetParticipantRole()
......................................................................
[consensus] KUDU-2839 fix assert in GetParticipantRole()
Before this patch, the consistency check in GetParticipantRole() was
incorrect. It tried to search for the replica UUID against pending
config, but replica UUID comes from committed config at the only call
site of the function in CatalogManager::BuildLocationsForTablet().
The pending Raft config might have replica already removed in case if
there is move/replace replica operation in progress. The incorrect
assertion triggered here:
http://dist-test.cloudera.org/job?job_id=aserbin.1577145505.5646
I also did a minor clean up of the code around.
This is a follow-up to 586e957f76a547340f2ab93a7eebc3f116ff825e.
Change-Id: I1a490f5b61dda8cebb8ee9bd9171ef3aa733a148
---
M src/kudu/consensus/quorum_util.cc
M src/kudu/consensus/quorum_util.h
M src/kudu/master/catalog_manager.cc
M src/kudu/master/catalog_manager.h
4 files changed, 12 insertions(+), 14 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/46/14946/2
--
To view, visit http://gerrit.cloudera.org:8080/14946
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I1a490f5b61dda8cebb8ee9bd9171ef3aa733a148
Gerrit-Change-Number: 14946
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
