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.

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

Trailing slashes cause S3 binding to throw NotATable #62

Closed
rtyler opened this issue Jan 10, 2021 · 0 comments · Fixed by #78
Closed

Trailing slashes cause S3 binding to throw NotATable #62

rtyler opened this issue Jan 10, 2021 · 0 comments · Fixed by #78
Labels
binding/rust Issues for the Rust crate bug Something isn't working good first issue Good for newcomers storage/aws AWS S3 storage related

Comments

@rtyler
Copy link
Member

rtyler commented Jan 10, 2021

The difference between deltalake::open_table("s3://mybucket/path/") and deltalake::open_table("s3://mybucket/path") is that the former throws an error, the latter succeeds.

IMHO Delta-rs shouldn't be so silly about redundant slashes.

@rtyler rtyler added bug Something isn't working binding/rust Issues for the Rust crate storage/aws AWS S3 storage related labels Jan 10, 2021
@houqp houqp added the good first issue Good for newcomers label Jan 11, 2021
@houqp houqp closed this as completed in #78 Jan 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
binding/rust Issues for the Rust crate bug Something isn't working good first issue Good for newcomers storage/aws AWS S3 storage related
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants