GitHub user gengliangwang opened a pull request:

    https://github.com/apache/spark/pull/21001

    [SPARK-19724][SQL][FOLLOW-UP]Check location of managed table when 
ignoreIfExists is true

    ## What changes were proposed in this pull request?
    
    In the PR #20886, I mistakenly check the table location only when 
`ignoreIfExists` is false, which was following the original deprecated PR.
    That was wrong. When `ignoreIfExists` is true and the target table doesn't 
exist, we should also check the table location. In other word, 
**`ignoreIfExists` has nothing to do with table location validation**.
    This is a follow-up PR to fix the mistake. 
    
    ## How was this patch tested?
    
    Add one unit test.


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/gengliangwang/spark SPARK-19724-followup

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/21001.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #21001
    
----
commit 3d1c90960f88042c51012f1b4df8eaffb73994c8
Author: Gengliang Wang <gengliang.wang@...>
Date:   2018-04-06T17:37:55Z

    SPARK-19724 follow up to fix ignoreIfExists branch

----


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to