Hello Tidy Bot, Kudu Jenkins, Adar Dembo,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/8254
to look at the new patch set (#5).
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
---
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(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/54/8254/5
--
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: newpatchset
Gerrit-Change-Id: I84ca3b207da28cd7dc43a077736da9b4e0ec6f37
Gerrit-Change-Number: 8254
Gerrit-PatchSet: 5
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
