Riza Suminto has posted comments on this change. ( http://gerrit.cloudera.org:8080/21491 )
Change subject: IMPALA-13143: Fix flaky test_catalogd_failover_with_sync_ddl ...................................................................... Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/21491/4/tests/custom_cluster/test_catalogd_ha.py File tests/custom_cluster/test_catalogd_ha.py: http://gerrit.cloudera.org:8080/#/c/21491/4/tests/custom_cluster/test_catalogd_ha.py@486 PS4, Line 486: 45 Change this to use SYNC_DDL_DELAY_S? Am I correct that test should wait for 2 * SYNC_DDL_DELAY_S + <estimated_query_time> since query start? 2 * SYNC_DDL_DELAY_S for two delays happen in first and second catalogd. -- To view, visit http://gerrit.cloudera.org:8080/21491 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I15bb6aae23a2f544067f993533e322969372ebd5 Gerrit-Change-Number: 21491 Gerrit-PatchSet: 4 Gerrit-Owner: Wenzhe Zhou <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Reviewer: Wenzhe Zhou <[email protected]> Gerrit-Comment-Date: Fri, 07 Jun 2024 17:58:37 +0000 Gerrit-HasComments: Yes
