-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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.
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
DO NOT MERGE: Testing the MacOS wheel failures #39567
DO NOT MERGE: Testing the MacOS wheel failures #39567
Conversation
Thanks for opening a pull request! If this is not a minor PR. Could you open an issue for this pull request on GitHub? https://github.com/apache/arrow/issues/new/choose Opening GitHub issues ahead of time contributes to the Openness of the Apache Arrow project. Then could you also rename the pull request title in the following format?
or
In the case of PARQUET issues on JIRA the title also supports:
See also: |
@github-actions crossbow submit wheel-macos |
Revision: bb48604 Submitted crossbow builds: ursacomputing/crossbow @ actions-512b0e912c |
So it's clearly not related to the logging fs wrapper around the filesystem .. |
@github-actions crossbow submit wheel-macos |
Revision: a60c5b3 Submitted crossbow builds: ursacomputing/crossbow @ actions-c22db1a11f |
@github-actions crossbow submit wheel-macos |
Revision: a60c5b3 Submitted crossbow builds: ursacomputing/crossbow @ actions-3bd0891f6d |
@github-actions crossbow submit wheel-macos |
Revision: 86c3018 Submitted crossbow builds: ursacomputing/crossbow @ actions-2cf7237b8b |
@github-actions crossbow submit wheel-macos |
Revision: 86c3018 Submitted crossbow builds: ursacomputing/crossbow @ actions-99f7730303 |
So I could nail down the failure to the following: arrow/cpp/src/arrow/dataset/file_parquet.cc Lines 814 to 823 in ac50918
In the above snippet, sometimes But I have no idea why that would sometimes return -1, and only on MacOS when running the test from an installed wheel (not any of the other Mac builds where we build Arrow directly) |
Fixed in #39632. We can close now. |
Test for #39562