Dan Burkert has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8254 )
Change subject: catalog manager: fix DDL race ...................................................................... catalog manager: fix DDL race This fixes a race between concurrent ALTER TABLE RENAME and DROP TABLE DDL operations. Previously it was possible for two such operations to concurrently complete on the same table. The included test is about 50% flaky without the fix. In addition, it changes the table finding logic to use the table ID and table name if both are present in the TableIdentifier. Previously only the table ID would be used in this case. No clients currently send both, so this shouldn't have any real effect, but I think it's useful to be able to specify both in certain cases. Change-Id: I84ca3b207da28cd7dc43a077736da9b4e0ec6f37 Reviewed-on: http://gerrit.cloudera.org:8080/8254 Tested-by: Kudu Jenkins Reviewed-by: Dan Burkert <[email protected]> --- M src/kudu/master/catalog_manager.cc M src/kudu/master/catalog_manager.h M src/kudu/master/master-test.cc M src/kudu/util/cow_object.h 4 files changed, 225 insertions(+), 65 deletions(-) Approvals: Kudu Jenkins: Verified Dan Burkert: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/8254 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I84ca3b207da28cd7dc43a077736da9b4e0ec6f37 Gerrit-Change-Number: 8254 Gerrit-PatchSet: 7 Gerrit-Owner: Dan Burkert <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot
