Skip to content

FileIO must use absolute paths.#2248

Closed
rambleraptor wants to merge 1 commit intoapache:mainfrom
rambleraptor:absolute_path
Closed

FileIO must use absolute paths.#2248
rambleraptor wants to merge 1 commit intoapache:mainfrom
rambleraptor:absolute_path

Conversation

@rambleraptor
Copy link
Copy Markdown
Contributor

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?

@kevinjqliu
Copy link
Copy Markdown
Contributor

I think it makes more sense to override the LocalFilesystem implementation in both fsspec and pyarrow instead of in FileIO. WDYT?

@ndrluis
Copy link
Copy Markdown
Collaborator

ndrluis commented Jan 12, 2026

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.

@github-actions
Copy link
Copy Markdown

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.

@github-actions github-actions bot added the stale label Mar 18, 2026
@github-actions
Copy link
Copy Markdown

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.

@github-actions github-actions bot closed this Mar 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ensure absolute path when referencing any file paths

3 participants