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