Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Adar Dembo, Hao Hao,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/13315
to look at the new patch set (#2).
Change subject: WIP hms: ignore DROP TABLE events from external tables
......................................................................
WIP hms: ignore DROP TABLE events from external tables
To maintain the expectation that dropping an external table will not
delete the underlying data of a Kudu table, this patch updates the
HMS notification log listener to ignore external tables.
To maintain the behavior that Kudu tables that are synchronized with the
HMS can continue to be dropped, this patch also sets the default table
type to be managed.
WIP: I looped master-stress-test with 7 stress threads and with this
patch, it failed 34/500 times.
This change makes it difficult to delete the underlying Kudu table of an
external table. A follow-up patch will provide the semantics described
in the new disabled test TestDropExternalTableThroughKudu.
Change-Id: I9b593ac40022d881569c524ac8f3c23b6f8815dc
---
M src/kudu/hms/hms_catalog.h
M src/kudu/integration-tests/hms_itest-base.cc
M src/kudu/integration-tests/hms_itest-base.h
M src/kudu/integration-tests/master_hms-itest.cc
M src/kudu/master/hms_notification_log_listener.cc
M src/kudu/tools/kudu-tool-test.cc
M src/kudu/tools/tool_action_hms.cc
7 files changed, 86 insertions(+), 12 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/15/13315/2
--
To view, visit http://gerrit.cloudera.org:8080/13315
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I9b593ac40022d881569c524ac8f3c23b6f8815dc
Gerrit-Change-Number: 13315
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Hao Hao <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
