Sai Hemanth Gantasala has posted comments on this change. ( http://gerrit.cloudera.org:8080/19155 )
Change subject: IMPALA-11626: Handle COMMIT_COMPACTION_EVENT from HMS ...................................................................... Patch Set 15: (5 comments) http://gerrit.cloudera.org:8080/#/c/19155/14//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/19155/14//COMMIT_MSG@13 PS14, Line 13: processing > nit: processing Ack http://gerrit.cloudera.org:8080/#/c/19155/14/fe/src/main/java/org/apache/impala/catalog/events/MetastoreEvents.java File fe/src/main/java/org/apache/impala/catalog/events/MetastoreEvents.java: http://gerrit.cloudera.org:8080/#/c/19155/14/fe/src/main/java/org/apache/impala/catalog/events/MetastoreEvents.java@2664 PS14, Line 2664: msTbl_ = tbl.getMetaStoreTable(); > Just realized that msTbl_ could be null which will break MetastoreTableEven Ack http://gerrit.cloudera.org:8080/#/c/19155/14/fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java File fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java: http://gerrit.cloudera.org:8080/#/c/19155/14/fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java@4524 PS14, Line 4524: returns 0 > nit: returns the number of partitions reloaded The above line already mentions that. so this change is not required. http://gerrit.cloudera.org:8080/#/c/19155/14/tests/custom_cluster/test_events_custom_configs.py File tests/custom_cluster/test_events_custom_configs.py: http://gerrit.cloudera.org:8080/#/c/19155/14/tests/custom_cluster/test_events_custom_configs.py@275 PS14, Line 275: test_cc_part_table = "test_cc_partitioned_table" > stale comment? We just insert to the save partition. Ack http://gerrit.cloudera.org:8080/#/c/19155/14/tests/custom_cluster/test_events_custom_configs.py@339 PS14, Line 339: > Can we test the case when the table is unloaded? Just want to cover the cas Ack -- To view, visit http://gerrit.cloudera.org:8080/19155 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I464faedb4e3bbcd417bab2e3cb0d57e339d42605 Gerrit-Change-Number: 19155 Gerrit-PatchSet: 15 Gerrit-Owner: Sai Hemanth Gantasala <saihema...@cloudera.com> Gerrit-Reviewer: Daniel Becker <daniel.bec...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com> Gerrit-Reviewer: Sai Hemanth Gantasala <saihema...@cloudera.com> Gerrit-Comment-Date: Tue, 28 Feb 2023 18:57:53 +0000 Gerrit-HasComments: Yes