You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/05/13 10:25:45 UTC

[GitHub] [arrow] raulcd commented on pull request #13147: ARROW-16548: [Python] Add pytest.mark.parquet to all tests under tests/parquet package

raulcd commented on PR #13147:
URL: https://github.com/apache/arrow/pull/13147#issuecomment-1125893203

   @jorisvandenbossche  other solutions that fix this issue could be:
   * Add the `pytestmark = pytest.mark.parquet` to each test file individually instead of to the common file
   * Add the `pytestmark` import (even when we are not using it) on each individual test file (`from pyarrow.tests.parquet.common import pytestmark`)
   
   I was going in a rabbit hole on importlib and load_module but I don't think is worth the effort to spend more time on this having three possible solutions.
   One nice thing is I had to go back to [a talk I gave at PyCon Spain](https://github.com/raulcd/import_magic/blob/master/magic_import_v2.ipynb) on how import works :)


-- 
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-unsubscribe@arrow.apache.org

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