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>