Michael Smith has posted comments on this change. ( http://gerrit.cloudera.org:8080/21905 )
Change subject: IMPALA-11761: Run large dir tests serially ...................................................................... Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/21905/1/tests/metadata/test_recursive_listing.py File tests/metadata/test_recursive_listing.py: http://gerrit.cloudera.org:8080/#/c/21905/1/tests/metadata/test_recursive_listing.py@171 PS1, Line 171: @pytest.mark.execute_serially > It is. I may have partially misunderstood "stress" the first time. Done. Although I'm skeptical the tests we have marked stress were actually written with high concurrent test execution in mind. They mostly look like individual tests that are meant to be a mini stress test on their own. https://github.com/apache/impala/blob/master/tests/stress/test_ddl_stress.py is the only test that actually seems to be written such that it expects to have multiple instances running at once. I suspect everything else currently marked 'stress' should actually be marked 'execute_serially'. -- To view, visit http://gerrit.cloudera.org:8080/21905 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I1f7d2017ae1bab0f2f8cb0b100c2c6cc8b4f3dcd Gerrit-Change-Number: 21905 Gerrit-PatchSet: 3 Gerrit-Owner: Michael Smith <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Comment-Date: Mon, 21 Oct 2024 17:43:53 +0000 Gerrit-HasComments: Yes
