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: github-unsubscr...@beam.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to