Impala Public Jenkins has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/14385 )
Change subject: IMPALA-9017: Alter table events on dropped table/db should be
ignored.
......................................................................
IMPALA-9017: Alter table events on dropped table/db should be
ignored.
Alter table events received on a table/db that does not exist in
catalog can be safely ignored. This is because a table not present
in the catalog means either:
1. The table was created in HMS but not processed by the events
processor beacuase it is already in an error state or,
2. It was dropped/renamed by the same catalog.
Prior to this change, alter table events generated by renames would
try to add the new table names to the cache irrespective of weather
old table was in catalog cache. After this change a drop + create
sequence happens only if table before rename exists in catalog.
Testing:
Added tests for rename + drop table/db sequences to verify events
processor does not error out.
Change-Id: I534accd704f76772ebe47865eff0fcdc7a033a48
Reviewed-on: http://gerrit.cloudera.org:8080/14385
Reviewed-by: Vihang Karajgaonkar <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
---
M fe/src/main/java/org/apache/impala/catalog/CatalogServiceCatalog.java
M fe/src/main/java/org/apache/impala/catalog/events/MetastoreEvents.java
M
fe/src/test/java/org/apache/impala/catalog/events/MetastoreEventsProcessorTest.java
3 files changed, 47 insertions(+), 30 deletions(-)
Approvals:
Vihang Karajgaonkar: Looks good to me, approved
Impala Public Jenkins: Verified
--
To view, visit http://gerrit.cloudera.org:8080/14385
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I534accd704f76772ebe47865eff0fcdc7a033a48
Gerrit-Change-Number: 14385
Gerrit-PatchSet: 3
Gerrit-Owner: Anurag Mantripragada <[email protected]>
Gerrit-Reviewer: Anurag Mantripragada <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Vihang Karajgaonkar <[email protected]>
