Conversation
|
I think it makes more sense to override the LocalFilesystem implementation in both fsspec and pyarrow instead of in FileIO. WDYT? |
|
Hey @rambleraptor, @kevinjqliu left a question that's still open. Do you have time to respond or update the PR? The issue had gone stale, so just wanted to bump this. |
|
This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that's incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@iceberg.apache.org list. Thank you for your contributions. |
|
This pull request has been closed due to lack of activity. This is not a judgement on the merit of the PR in any way. It is just a way of keeping the PR queue manageable. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time. |
Closes #1730
Rationale for this change
Ensure that FileIO must use absolute paths.
Are these changes tested?
Unit test fixes.
Are there any user-facing changes?