Adar Dembo has submitted this change and it was merged. Change subject: sys_catalog: put the correct tablet ID in delete catalog table requests ......................................................................
sys_catalog: put the correct tablet ID in delete catalog table requests I don't think this has any real effect because we only ever delete the catalog table in unit tests. And even if those tests used multiple masters, I can't see where a follower would use the tablet ID from the WriteRequestPB and not from the ConsensusRequestPB. Change-Id: I5a3f56fea5f6f590f1bc76c32fe0a44cffe9cfcc Reviewed-on: http://gerrit.cloudera.org:8080/2369 Tested-by: Kudu Jenkins Reviewed-by: Todd Lipcon <[email protected]> --- M src/kudu/master/sys_catalog.cc 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Todd Lipcon: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/2369 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5a3f56fea5f6f590f1bc76c32fe0a44cffe9cfcc Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
