Jason Fehr has posted comments on this change. ( http://gerrit.cloudera.org:8080/22119 )
Change subject: IMPALA-13536: Fix Workload Management Init Tests Issues ...................................................................... Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/22119/2/tests/common/custom_cluster_test_suite.py File tests/common/custom_cluster_test_suite.py: http://gerrit.cloudera.org:8080/#/c/22119/2/tests/common/custom_cluster_test_suite.py@510 PS2, Line 510: LOG.info("Log Files for Test:\n{}".format(file_list)) > Do you mean when test fail or when start-impala-cluster.py fail? The code s When start-impala-cluster.py runs, it removes all existing symlinks (at least that is what it did yesterday on my local dev machine when I tested it). I was thinking more about this topic last night, and I agree that printing the resolved symlinks will create noise as the majority of tests don't need to trace errors across multiple log files. I created a new parameter on CustomClusterTestSuite that will log the symlinks if specified. http://gerrit.cloudera.org:8080/#/c/22119/2/tests/custom_cluster/test_workload_mgmt_init.py File tests/custom_cluster/test_workload_mgmt_init.py: http://gerrit.cloudera.org:8080/#/c/22119/2/tests/custom_cluster/test_workload_mgmt_init.py@193 PS2, Line 193: cluster_size=10 > Seems unnecessary to start 10 nodes to only run the test file against just I think it's a good idea to connect to different coordinators. I modified the code to randomly choose a coordinator in situations where more than 1 coordinator is configured. -- To view, visit http://gerrit.cloudera.org:8080/22119 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie3f34088d1d925f30abb63471387e6fdb62b95a7 Gerrit-Change-Number: 22119 Gerrit-PatchSet: 2 Gerrit-Owner: Jason Fehr <[email protected]> Gerrit-Reviewer: Abhishek Rawat <[email protected]> Gerrit-Reviewer: Andrew Sherman <[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: Tue, 26 Nov 2024 20:30:48 +0000 Gerrit-HasComments: Yes
