Riza Suminto has posted comments on this change. ( http://gerrit.cloudera.org:8080/22065 )
Change subject: Improve error message in EE tests after crash ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/22065/1/tests/common/impala_test_suite.py File tests/common/impala_test_suite.py: http://gerrit.cloudera.org:8080/#/c/22065/1/tests/common/impala_test_suite.py@252 PS1, Line 252: expected_impalads = 3 > CustomClusterTestSuite overrides setup_class, so this change will have no e There is a call to super().setup_class() at https://gerrit.cloudera.org/c/22006/9/tests/common/custom_cluster_test_suite.py#322 Also, in case of EE test, 3 can be derived from len(IMPALAD_HOST_PORT_LIST)? -- To view, visit http://gerrit.cloudera.org:8080/22065 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I17cce8ccda85c9f5e4b1288e1708c8b84ea724f1 Gerrit-Change-Number: 22065 Gerrit-PatchSet: 1 Gerrit-Owner: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Comment-Date: Wed, 13 Nov 2024 16:12:27 +0000 Gerrit-HasComments: Yes
