udim commented on a change in pull request #15104:
URL: https://github.com/apache/beam/pull/15104#discussion_r661724487
##########
File path: sdks/python/apache_beam/options/pipeline_options_test.py
##########
@@ -215,6 +216,7 @@ def _add_argparse_args(cls, parser):
parser.add_argument(
'--fake_multi_option', action='append', help='fake multi option')
+ @pytest.mark.no_xdist
Review comment:
no_xdist affects how the test is run by run_pytest.sh. The script was
written because we have some tests that don't work well with xdist. Flakiness
though isn't a good reason to disable xdist.
I think that this might be the culprit:
https://github.com/apache/beam/blob/07bf18a75f3e4b28cd496b83b5aca9bea34f78e1/sdks/python/apache_beam/runners/worker/sdk_worker_main_test.py#L93
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
