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

[SPARK-19329][SQL][BRANCH-2.1]Reading from or writing to a datasource table with a non pre-existing location should succeed #17317

Closed
wants to merge 2 commits into from
Closed

[SPARK-19329][SQL][BRANCH-2.1]Reading from or writing to a datasource table with a non pre-existing location should succeed #17317

wants to merge 2 commits into from

Conversation

windpiger
Copy link
Contributor

What changes were proposed in this pull request?

This is a backport pr of #16672 into branch-2.1.

How was this patch tested?

Existing tests.

@windpiger
[SPARK-19329][SQL][BACKPORT-2.1]backport to branch-2.1,Reading from o…
37eb1dc 
…r writing to a datasource table with a non pre-existing location should succeed
@windpiger
fix some code
96c7678
@windpiger
Copy link
Contributor Author

windpiger commented Mar 16, 2017 •
edited

cc @gatorsmile I have backport to branch-2.1 which you need~

@SparkQA
Copy link

SparkQA commented Mar 16, 2017

Test build #74671 has finished for PR 17317 at commit 96c7678.

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

gatorsmile
gatorsmile approved these changes Mar 16, 2017
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

@gatorsmile
Copy link
Member

Thanks! Merging to 2.1.

asfgit pushed a commit that referenced this pull request Mar 16, 2017
@windpiger @gatorsmile
[SPARK-19329][SQL][BRANCH-2.1] Reading from or writing to a datasourc…
9d032d0 
…e table with a non pre-existing location should succeed

## What changes were proposed in this pull request?

This is a backport pr of #16672 into branch-2.1.

## How was this patch tested?
Existing tests.

Author: windpiger <songjun@outlook.com>

Closes #17317 from windpiger/backport-insertnotexists.
@gatorsmile
Copy link
Member

@windpiger Could you please close it? Thanks!

@windpiger windpiger closed this Mar 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gatorsmile gatorsmile gatorsmile approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@windpiger @SparkQA @gatorsmile