Sai Hemanth Gantasala has posted comments on this change. ( http://gerrit.cloudera.org:8080/23487 )
Change subject: IMPALA-14131: Add flag to configure the default value of 'impala.disableHmsSync' ...................................................................... Patch Set 10: (3 comments) http://gerrit.cloudera.org:8080/#/c/23487/9/tests/custom_cluster/test_events_custom_configs.py File tests/custom_cluster/test_events_custom_configs.py: http://gerrit.cloudera.org:8080/#/c/23487/9/tests/custom_cluster/test_events_custom_configs.py@1855 PS9, Line 1855: assert events_skipped_after == events_skipped_before + skip_events, \ > nit: this is stale when 'skip_events' is not 0 Ack http://gerrit.cloudera.org:8080/#/c/23487/9/tests/custom_cluster/test_events_custom_configs.py@1857 PS9, Line 1857: nt > Let's print 'skip_events' here. Ack http://gerrit.cloudera.org:8080/#/c/23487/9/tests/custom_cluster/test_events_custom_configs.py@1885 PS9, Line 1885: .format(tbl2)) > nit: can we also test tbl2 here? Ack -- To view, visit http://gerrit.cloudera.org:8080/23487 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I4ee617aed48575502d9cf5cf2cbea6ec897d6839 Gerrit-Change-Number: 23487 Gerrit-PatchSet: 10 Gerrit-Owner: Sai Hemanth Gantasala <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Sai Hemanth Gantasala <[email protected]> Gerrit-Comment-Date: Wed, 10 Dec 2025 22:31:42 +0000 Gerrit-HasComments: Yes
