Riza Suminto has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/22571 )

Change subject: IMPALA-13684: Improve waitForHmsEvent() to only wait for 
related events
......................................................................


Patch Set 13: Code-Review+1

(2 comments)

http://gerrit.cloudera.org:8080/#/c/22571/12/tests/custom_cluster/test_events_custom_configs.py
File tests/custom_cluster/test_events_custom_configs.py:

http://gerrit.cloudera.org:8080/#/c/22571/12/tests/custom_cluster/test_events_custom_configs.py@1624
PS12, Line 1624:       
catalogd_args="--debug_actions=catalogd_event_processing_delay:SLEEP@200")
               :   def test_hms_event_sync_with_commit_compaction(self, vector, 
un
> Good point!
Done


http://gerrit.cloudera.org:8080/#/c/22571/12/tests/metadata/test_event_processing.py
File tests/metadata/test_event_processing.py:

http://gerrit.cloudera.org:8080/#/c/22571/12/tests/metadata/test_event_processing.py@414
PS12, Line 414:     self.run_stmt_in_hive(
              :         "insert overwrite table {0} partition (p=0) select 
> Done
There are more pattern like this below. Would you like to change it?



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic033b7e197cd19505653c3ff80c4857cc474bcfc
Gerrit-Change-Number: 22571
Gerrit-PatchSet: 13
Gerrit-Owner: Quanlong Huang <huangquanl...@gmail.com>
Gerrit-Reviewer: Anonymous Coward <k.venureddy2...@gmail.com>
Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Jason Fehr <jf...@cloudera.com>
Gerrit-Reviewer: Kurt Deschler <kdesc...@cloudera.com>
Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com>
Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com>
Gerrit-Reviewer: Sai Hemanth Gantasala <saihema...@cloudera.com>
Gerrit-Comment-Date: Thu, 17 Apr 2025 02:11:55 +0000
Gerrit-HasComments: Yes

Reply via email to