Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Python][Parquet][Emscripten] PARQUET_TEST_DATA isn't set in test-conda-python-emscripten #43868

Closed
kou opened this issue Aug 29, 2024 · 1 comment
Closed

[Python][Parquet][Emscripten] PARQUET_TEST_DATA isn't set in test-conda-python-emscripten #43868

kou opened this issue Aug 29, 2024 · 1 comment
Assignees
@raulcd
Labels
Component: Parquet Component: Python Type: bug
Milestone
18.0.0

Comments

@kou
Copy link
Member

kou commented Aug 29, 2024

Describe the bug, including details regarding any error messages, version, and platform.

It's required since #41580.

https://github.com/ursacomputing/crossbow/actions/runs/10604493737/job/29391275377#step:7:8847

 ==================================== ERRORS ====================================
____________ ERROR at setup of test_column_chunk_key_value_metadata ____________

    @pytest.fixture(scope='module')
    def parquet_test_datadir():
        result = os.environ.get('PARQUET_TEST_DATA')
        if not result:
>           raise RuntimeError('Please point the PARQUET_TEST_DATA environment '
                               'variable to the test data directory')
E           RuntimeError: Please point the PARQUET_TEST_DATA environment variable to the test data directory

/lib/python3.12/site-packages/pyarrow/tests/parquet/conftest.py:35: RuntimeError

Component(s)

Parquet, Python
@kou kou added the Type: bug label Aug 29, 2024
@kou kou mentioned this issue Aug 29, 2024
Merged
@apache apache deleted a comment Aug 29, 2024
@kou kou mentioned this issue Sep 3, 2024
Closed
@github-actions github-actions bot mentioned this issue Sep 3, 2024
Merged
kou pushed a commit that referenced this issue Sep 18, 2024
@raulcd @jorisvandenbossche
GH-43868: [CI][Python] Skip test that requires PARQUET_TEST_DATA env …
fe39c8f 
…on emscripten (#43906)

### Rationale for this change

The following PR:
- #41580

Made mandatory for a test the requirement to have `PARQUET_TEST_DATA` env defined.

This is currently not available from `python_test_emscripten.sh` as we require to mount the filesystem for both Node and ChromeDriver.

### What changes are included in this PR?

Skip the test that requires `PARQUET_TEST_DATA` for emscripten.

### Are these changes tested?

Via archery

### Are there any user-facing changes?

No
* GitHub Issue: #43905
* GitHub Issue: #43868

Lead-authored-by: Raúl Cumplido <raulcumplido@gmail.com>
Co-authored-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
@kou
Copy link
Member Author

kou commented Sep 18, 2024

Issue resolved by pull request 43906
#43906

@kou kou added this to the 18.0.0 milestone Sep 18, 2024
@kou kou closed this as completed Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@raulcd raulcd

Labels
Component: Parquet Component: Python Type: bug
Projects
None yet
Milestone
18.0.0
Development

No branches or pull requests
2 participants
@kou @raulcd