Mike Percy has submitted this change and it was merged. Change subject: KUDU-2141. master: Remove DCHECK when tablet report has no opid_index ......................................................................
KUDU-2141. master: Remove DCHECK when tablet report has no opid_index Commit 2108767bf5331e0f3beccd56a987cb413cca380a made it possible for a tombstoned replica to report an empty committed config with no opid_index associated with it. The master should handle such a tablet by simply ignoring it. The included modification to tombstoned_voting-itest.cc causes the test to fail without the included changes to catalog_manager.cc Change-Id: Ia99936b11b49a4bd70dbc065f2d16136eb5b8bda Reviewed-on: http://gerrit.cloudera.org:8080/8019 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Kudu Jenkins Reviewed-by: Andrew Wong <[email protected]> --- M src/kudu/integration-tests/tombstoned_voting-itest.cc M src/kudu/master/catalog_manager.cc 2 files changed, 31 insertions(+), 10 deletions(-) Approvals: Andrew Wong: Looks good to me, but someone else must approve Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/8019 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia99936b11b49a4bd70dbc065f2d16136eb5b8bda Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Mike Percy <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]>
