Csaba Ringhofer has posted comments on this change. ( http://gerrit.cloudera.org:8080/18537 )
Change subject: IMPALA-7864: (Addendum) Deflake test_replan_limit by postponing catalog fetches ...................................................................... Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/18537/2/tests/custom_cluster/test_local_catalog.py File tests/custom_cluster/test_local_catalog.py: http://gerrit.cloudera.org:8080/#/c/18537/2/tests/custom_cluster/test_local_catalog.py@276 PS2, Line 276: print 'Attempt', attempt, 'client', str(client) Is this intentional, or just debug code? http://gerrit.cloudera.org:8080/#/c/18537/2/tests/custom_cluster/test_local_catalog.py@292 PS2, Line 292: t.join(600) Will we usually need this much time? 10 mins looks a lot, especially if the test is serialized -- To view, visit http://gerrit.cloudera.org:8080/18537 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia5bdca7402039f1f24b7bf19595c2541fa32d0ad Gerrit-Change-Number: 18537 Gerrit-PatchSet: 2 Gerrit-Owner: Quanlong Huang <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Comment-Date: Mon, 23 May 2022 07:04:24 +0000 Gerrit-HasComments: Yes
