You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@beam.apache.org by "Brian Hulette (Jira)" <ji...@apache.org> on 2021/09/29 21:52:00 UTC

[jira] [Commented] (BEAM-12985) pyarrow tests are executed twice in py38-pyarrow-x tox task

    [ https://issues.apache.org/jira/browse/BEAM-12985?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17422413#comment-17422413 ] 

Brian Hulette commented on BEAM-12985:
--------------------------------------

Based on [this issue|https://github.com/pytest-dev/pytest/issues/9121], it seems the -m behavior is intentional.

> pyarrow tests are executed twice in py38-pyarrow-x tox task
> -----------------------------------------------------------
>
>                 Key: BEAM-12985
>                 URL: https://issues.apache.org/jira/browse/BEAM-12985
>             Project: Beam
>          Issue Type: Bug
>          Components: sdk-py-core, testing
>            Reporter: Brian Hulette
>            Priority: P2
>
> {{tox -e py37-pyarrow-2}} runs all tests with the "uses_pyarrow" mark twice. This is because run_pytest.sh [runs pytest twice|https://github.com/apache/beam/blob/6beeafff496f69499cdb14bd58c6ac2d9e84d116/sdks/python/scripts/run_pytest.sh#L33], once with {{-m no_xdist}} and once with {{-m 'not no_xdist'}}. The pyarrow task uses [-m uses_pyarrow|https://github.com/apache/beam/blob/6beeafff496f69499cdb14bd58c6ac2d9e84d116/sdks/python/tox.ini#L249] to select tests, but pytest only respects the latest -m argument. So the no_xdist parameters are ignored, and _all_ pyarrow tests are run each time.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)