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

Change subject: IMPALA-13500: Fix test_invalidate_stale_partition_on_reload() 
test
......................................................................


Patch Set 4:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/22084/4/tests/custom_cluster/test_events_custom_configs.py
File tests/custom_cluster/test_events_custom_configs.py:

http://gerrit.cloudera.org:8080/#/c/22084/4/tests/custom_cluster/test_events_custom_configs.py@1327
PS4, Line 1327:     time.sleep(5)  # file metadata reload runs asynchronously, 
so wait for 2s
Lets drop this sleep(5) and increase timeout_s=20 in L1331.
It can speed up the test if log_regex to shows up in less than 5s 
(assert_impalad_log_contains check every 1 second).



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I62170aa6ed8ae122482a03212fec9c4fe843ce03
Gerrit-Change-Number: 22084
Gerrit-PatchSet: 4
Gerrit-Owner: Sai Hemanth Gantasala <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>
Gerrit-Reviewer: Sai Hemanth Gantasala <[email protected]>
Gerrit-Comment-Date: Tue, 26 Nov 2024 03:15:31 +0000
Gerrit-HasComments: Yes

Reply via email to