Vihang Karajgaonkar has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17576 )
Change subject: IMPALA-10746: Drop table/db from catalog cache when drop table/db HMS apis are accessed from catalog's metastore server. ...................................................................... IMPALA-10746: Drop table/db from catalog cache when drop table/db HMS apis are accessed from catalog's metastore server. This patch fixes a scenario where if table/db already exists in cache and a user drops it via catalog metastore server endpoint (i.e drop_table HMS api). When recreating the same via Impala Shell, user gets an error that table/db already exists. The patch fixes it by dropping table/db from HMS endpoints so that new table/db succeeds Testing: Added new unit tests which cover drop_database and drop_table HMS API Change-Id: Ic2e2ad2630e2028b8ad26a6272ee766b27e0935c Reviewed-on: http://gerrit.cloudera.org:8080/17576 Reviewed-by: <[email protected]> Reviewed-by: Vihang Karajgaonkar <[email protected]> Tested-by: Vihang Karajgaonkar <[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/main/java/org/apache/impala/catalog/metastore/MetastoreServiceHandler.java M fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java M tests/custom_cluster/test_metastore_service.py M tests/util/event_processor_utils.py 6 files changed, 326 insertions(+), 48 deletions(-) Approvals: [email protected]: Looks good to me, but someone else must approve Vihang Karajgaonkar: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/17576 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ic2e2ad2630e2028b8ad26a6272ee766b27e0935c Gerrit-Change-Number: 17576 Gerrit-PatchSet: 23 Gerrit-Owner: Sourabh Goyal <[email protected]> Gerrit-Reviewer: Anonymous Coward <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Sourabh Goyal <[email protected]> Gerrit-Reviewer: Vihang Karajgaonkar <[email protected]> Gerrit-Reviewer: Yu-Wen Lai <[email protected]>
