Skip to content

[SPARK-19724][SQL] allowCreatingManagedTableUsingNonemptyLocation should have legacy prefix #22515

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
Closed
rxin wants to merge 2 commits into from
Closed

[SPARK-19724][SQL] allowCreatingManagedTableUsingNonemptyLocation should have legacy prefix #22515

rxin wants to merge 2 commits into from

Conversation

@rxin
Copy link
Contributor

@rxin rxin commented Sep 21, 2018

One more legacy config to go ...

@SparkQA
Copy link

SparkQA commented Sep 21, 2018

Test build #96402 has finished for PR 22515 at commit 0f32b01.

  • This patch fails due to an unknown error code, -9.
  • This patch merges cleanly.
  • This patch adds no public classes.

@dilipbiswal
Copy link
Contributor

dilipbiswal commented Sep 21, 2018

retest this please

@SparkQA
Copy link

SparkQA commented Sep 21, 2018

Test build #96412 has finished for PR 22515 at commit 0f32b01.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

gatorsmile
gatorsmile reviewed Sep 21, 2018
View reviewed changes
Copy link
Member

@gatorsmile gatorsmile left a comment

Choose a reason for hiding this comment

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

LGTM

Thanks! Merged to master/2.4

asfgit pushed a commit that referenced this pull request Sep 21, 2018
@rxin @gatorsmile
[SPARK-19724][SQL] allowCreatingManagedTableUsingNonemptyLocation sho…
ce66361 
…uld have legacy prefix

One more legacy config to go ...

Closes #22515 from rxin/allowCreatingManagedTableUsingNonemptyLocation.

Authored-by: Reynold Xin <[email protected]>
Signed-off-by: gatorsmile <[email protected]>
(cherry picked from commit 4a11209)
Signed-off-by: gatorsmile <[email protected]>
@asfgit asfgit closed this in 4a11209 Sep 21, 2018
@dongjoon-hyun dongjoon-hyun mentioned this pull request Oct 5, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gatorsmile gatorsmile gatorsmile left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@rxin @SparkQA @dilipbiswal @gatorsmile