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

[Bug]: Spark client panics when reading DataFrame from entire repository / storageNamespace #7955

Merged
merged 4 commits into from
Jul 4, 2024

Conversation

Isan-Rivkin
Copy link
Contributor

@Isan-Rivkin Isan-Rivkin commented Jul 3, 2024

Closes #7914

The issue was that resolve overrides the URI path completely.
So if metadataURI the value expected to be: s3a:https://<bucket>/<path>/_lakefs it instead became s3a:https://<bucket>/_lakefs.
When I was testing the issue there was such path so I saw very strange results and parsing errors.

namespaceURI.resolve(...)

@Isan-Rivkin Isan-Rivkin self-assigned this Jul 3, 2024
@Isan-Rivkin Isan-Rivkin added bug Something isn't working include-changelog PR description should be included in next release changelog labels Jul 3, 2024
Copy link

github-actions bot commented Jul 3, 2024

E2E Test Results - DynamoDB Local - Local Block Adapter

13 passed

Copy link
Contributor

@arielshaqed arielshaqed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@Isan-Rivkin Isan-Rivkin merged commit a9ec2d7 into master Jul 4, 2024
35 checks passed
@Isan-Rivkin Isan-Rivkin deleted the bug/7914-spark-md-df-listing branch July 4, 2024 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working include-changelog PR description should be included in next release changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Spark client panics when reading DataFrame from entire repository / storageNamespace
2 participants