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

Let connections.get_udf_bucket_path() return a Path object instead of a string #37

Open
ahsimb opened this issue Nov 30, 2023 · 2 comments
Open

Let connections.get_udf_bucket_path() return a Path object instead of a string #37

ahsimb opened this issue Nov 30, 2023 · 2 comments
Assignees
@ahsimb
Labels
refactoring Code improvement without behavior change

Comments

@ahsimb
Copy link
Collaborator

ahsimb commented Nov 30, 2023

No description provided.

@ahsimb ahsimb added the refactoring Code improvement without behavior change label Nov 30, 2023
@ahsimb ahsimb self-assigned this Nov 30, 2023
@ahsimb ahsimb mentioned this issue Nov 30, 2023
Merged
@tkilias
Copy link
Collaborator

tkilias commented Aug 26, 2024

@ahsimb I think, we should move the notebook-connector and ai-lab to the BucketFS-Python Pathlike interface that make the whole get_udf_bucket_path obsolete.

@ahsimb
Copy link
Collaborator Author

ahsimb commented Aug 27, 2024

This function returns the path as it's seen by a udf. That's to let a user read files with a standard File I/O.
I agree that using the Pathlike interface is better than opening a Bucket.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ahsimb ahsimb

Labels
refactoring Code improvement without behavior change
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tkilias @ahsimb