GitHub user cloud-fan opened a pull request:

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

    correct several partition related behaviours of ExternalCatalog

    ## What changes were proposed in this pull request?
    
    This PR corrects several partition related behaviors of `ExternalCatalog`:
    
    1. default table location should not always lower case the table name in 
path string(fix `HiveExternalCatalog`)
    2. default partition location should not always lower case the partition 
column names in path string(fix `HiveExternalCatalog`)
    3. rename partition should not always lower case the partition column names 
in updated partition path string(fix `HiveExternalCatalog`)
    4. rename partition should update the partition location only for managed 
table(fix `InMemoryCatalog`)
    5. create partition with existing directory should be fine(fix 
`InMemoryCatalog`)
    6. create partition with non-existing directory should create that 
directory(fix `InMemoryCatalog`)
    7. drop partition from external table should not delete the directory(fix 
`InMemoryCatalog`)
    
    ## How was this patch tested?
    
    new tests in `ExternalCatalogSuite`
    


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

    $ git pull https://github.com/cloud-fan/spark partition

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

    https://github.com/apache/spark/pull/15797.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 #15797
    
----
commit e0e012c95878f4a9343ed76c6629d667bd2c0074
Author: Wenchen Fan <wenc...@databricks.com>
Date:   2016-11-07T15:44:55Z

    correct several partition related behaviours of ExternalCatalog

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to