Jason Fehr has posted comments on this change. ( http://gerrit.cloudera.org:8080/22551 )
Change subject: IMPALA-13710: Fix Flaky Workload Management Custom Cluster Tests ...................................................................... Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/22551/2/tests/util/workload_management.py File tests/util/workload_management.py: http://gerrit.cloudera.org:8080/#/c/22551/2/tests/util/workload_management.py@73 PS2, Line 73: for cntr in range(3): > You can sleep(1) and loop until timeout reached, like Good idea. Also made the change in the other for loop in assert_csv_col. http://gerrit.cloudera.org:8080/#/c/22551/2/tests/util/workload_management.py@88 PS2, Line 88: 3000 > This is 3000s? I meant to sleep for 3 seconds and forgot python's sleep takes seconds as its input. -- To view, visit http://gerrit.cloudera.org:8080/22551 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I38dad4819190fa44ca2933be9907d7f1273621e3 Gerrit-Change-Number: 22551 Gerrit-PatchSet: 2 Gerrit-Owner: Jason Fehr <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jason Fehr <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Comment-Date: Wed, 26 Feb 2025 21:50:47 +0000 Gerrit-HasComments: Yes
