Wenzhe Zhou has posted comments on this change. ( http://gerrit.cloudera.org:8080/15984 )
Change subject: IMPALA-9775: Fix test failure in TestAcid.test_acid_heartbeats ...................................................................... Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/15984/1/fe/src/main/java/org/apache/impala/common/TransactionKeepalive.java File fe/src/main/java/org/apache/impala/common/TransactionKeepalive.java: http://gerrit.cloudera.org:8080/#/c/15984/1/fe/src/main/java/org/apache/impala/common/TransactionKeepalive.java@52 PS1, Line 52: // But we have to set sleep interval not beyond 60 seconds for carrying through the : // unit test case TestAcid.test_acid_heartbeats (IMPALA-9775). > Normally we should use the same "hive.txn.timeout" value that HMS uses to a Will change the upper limit to 100000 millisecond. http://gerrit.cloudera.org:8080/#/c/15984/1/tests/query_test/test_acid.py File tests/query_test/test_acid.py: http://gerrit.cloudera.org:8080/#/c/15984/1/tests/query_test/test_acid.py@177 PS1, Line 177: @pytest.mark.execute_serially > I think this can be removed. Agree, will remove it. http://gerrit.cloudera.org:8080/#/c/15984/1/tests/query_test/test_acid.py@191 PS1, Line 191: (sleep(200000) > Yeah, that's why I suggested above to remove the @pytest.mark.execute_seria Agree, will increase sleep time to 320000, and MAX_ATTEMPTS to 16. And skip the table formats with compression. With these changes, the execution time for this test case reduce to about 3 minutes. -- To view, visit http://gerrit.cloudera.org:8080/15984 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I7922797d7e3ce94a2c8948211245f4e77fdb08b7 Gerrit-Change-Number: 15984 Gerrit-PatchSet: 1 Gerrit-Owner: Wenzhe Zhou <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Reviewer: Wenzhe Zhou <[email protected]> Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]> Gerrit-Comment-Date: Tue, 26 May 2020 20:46:32 +0000 Gerrit-HasComments: Yes
