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

Append a slash to DeltaLake splits' path only when necessary #14727

Merged
merged 1 commit into from
Oct 26, 2022

Conversation

assaf2
Copy link
Member

@assaf2 assaf2 commented Oct 24, 2022

Description

Before this change, a slash was always appended to tableLocation, even when not necessary.
create table as with location='<s3_path>/' didn't fail, but the next SELECT failed on The specified key does not exist.
This doesn't happen on Iceberg, so I assume this is not intentional.

Release notes

( X) This is not user-visible or docs only and no release notes are required.
( ) Release notes are required, please propose a release note for me.
( ) Release notes are required, with the following suggested text:

@cla-bot cla-bot bot added the cla-signed label Oct 24, 2022
@assaf2 assaf2 requested review from jirassimok and ebyhr October 24, 2022 15:26
@assaf2 assaf2 force-pushed the append-slash-only-when-necessary branch from d6c8374 to 99e3767 Compare October 25, 2022 06:34
@assaf2 assaf2 requested a review from jirassimok October 25, 2022 06:36
Copy link
Member

@jirassimok jirassimok left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks!

@assaf2 assaf2 force-pushed the append-slash-only-when-necessary branch from 1ac4b96 to c9cdcc4 Compare October 26, 2022 07:14
@assaf2 assaf2 requested a review from ebyhr October 26, 2022 07:14
@assaf2 assaf2 self-assigned this Oct 26, 2022
@ebyhr ebyhr merged commit 1b934bf into trinodb:master Oct 26, 2022
@ebyhr
Copy link
Member

ebyhr commented Oct 26, 2022

Merged, thanks!

@github-actions github-actions bot added this to the 401 milestone Oct 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants