Todd Lipcon has submitted this change and it was merged. Change subject: TableInfo::RemoveTablet should use partition_key_start ......................................................................
TableInfo::RemoveTablet should use partition_key_start TableInfo::RemoveTablet try to remove tablet by tablet_id, but tablet_map_ uses partition_key_start as key, so it does not work. This commit fix this bug and add unit test for RemoveTablet. Change-Id: I3b34ea8d4e04dd8a8b567dfc451edeabe34c7e2b Reviewed-on: http://gerrit.cloudera.org:8080/1768 Reviewed-by: Todd Lipcon <[email protected]> Tested-by: Internal Jenkins --- M src/kudu/master/catalog_manager-test.cc M src/kudu/master/catalog_manager.cc M src/kudu/master/catalog_manager.h 3 files changed, 16 insertions(+), 12 deletions(-) Approvals: Internal Jenkins: Verified Todd Lipcon: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/1768 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3b34ea8d4e04dd8a8b567dfc451edeabe34c7e2b Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Binglin Chang <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Binglin Chang <[email protected]> Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]>
