Vihang Karajgaonkar has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/17707


Change subject: IMPALA-10815: Ignore events on non-default hive catalogs
......................................................................

IMPALA-10815: Ignore events on non-default hive catalogs

Hive-3 supports a new type in metastore called catalogs. Even
though impala does not support custom catalogs, it is still possible
that some external HMS client creates objects within a non-default
catalog. This can become problematic when the objects within
the custom catalog match with the name of other objects in the default
catalog. For example, dropping a custom catalog generates a
DROP_DATABASE event on default database of that catalog. When such
event is processed, the events processor can remove the default
database.

This patch adds logic to ignore all the events which are
generated on such non-default catalog objects. The default value of
catalog is defined in the hive-site.xml of the metastore client which
is used by catalogd. If the value is not present it default to "hive".

Additionally, it also adds the code to validate that the default
catalog name defined in the hive-site.xml of the catalogd is same
as in metastore server side. If the values do not match, the
events processor does not come up.

Testing:
1. Added a new test which creates a custom hive catalog and events
on it. The test makes sure that such events do not affect objects in
catalogd.

Change-Id: Ided463b2a98331d3d305bbe92fbf3a5d2e197acf
---
M fe/src/compat-hive-3/java/org/apache/impala/compat/MetastoreShim.java
M 
fe/src/main/java/org/apache/impala/catalog/events/MetastoreEventProcessorConfig.java
M fe/src/main/java/org/apache/impala/catalog/events/MetastoreEvents.java
M 
fe/src/main/java/org/apache/impala/catalog/events/MetastoreEventsProcessor.java
M 
fe/src/test/java/org/apache/impala/catalog/events/MetastoreEventsProcessorTest.java
5 files changed, 140 insertions(+), 35 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/07/17707/2
--
To view, visit http://gerrit.cloudera.org:8080/17707
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ided463b2a98331d3d305bbe92fbf3a5d2e197acf
Gerrit-Change-Number: 17707
Gerrit-PatchSet: 2
Gerrit-Owner: Vihang Karajgaonkar <[email protected]>

Reply via email to