Quanlong Huang has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/14296 )

Change subject: IMPALA-8968: Alter database events on dropped database should 
not put events processor in error state.
......................................................................

IMPALA-8968: Alter database events on dropped database should not put
events processor in error state.

This change is two-fold:
1. If an alter database event is received on database that does not
   exist, the event can be safely ignored. The events processor should
   only go into an error state if updateDb() fails.

2. This change also adds catalog service identifiers to create/drop
   function operations as Impala generates alter database events
   with these operations and they should be detected as self-events
   and ignored.

Testing:
Add tests to verify both the above changes to
MetastoreEventsProcessorTest.

Change-Id: I5f76136aeff35d1d38fc8b3d9a38da399d36eced
Reviewed-on: http://gerrit.cloudera.org:8080/14296
Reviewed-by: Quanlong Huang <huangquanl...@gmail.com>
Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
---
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/main/java/org/apache/impala/service/CatalogOpExecutor.java
M 
fe/src/test/java/org/apache/impala/catalog/events/MetastoreEventsProcessorTest.java
4 files changed, 110 insertions(+), 2 deletions(-)

Approvals:
  Quanlong Huang: Looks good to me, approved
  Impala Public Jenkins: Verified

--
To view, visit http://gerrit.cloudera.org:8080/14296
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I5f76136aeff35d1d38fc8b3d9a38da399d36eced
Gerrit-Change-Number: 14296
Gerrit-PatchSet: 8
Gerrit-Owner: Anurag Mantripragada <anu...@cloudera.com>
Gerrit-Reviewer: Anurag Mantripragada <anu...@cloudera.com>
Gerrit-Reviewer: Bharath Vissapragada <bhara...@apache.org>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com>

Reply via email to