{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/15797","repository_url":"https://api.github.com/repos/apache/spark","labels_url":"https://api.github.com/repos/apache/spark/issues/15797/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/15797/comments","events_url":"https://api.github.com/repos/apache/spark/issues/15797/events","html_url":"https://github.com/apache/spark/pull/15797","id":187751626,"number":15797,"title":"[SPARK-17990][SPARK-18302][SQL] correct several partition related behaviours of ExternalCatalog","user":{"login":"cloud-fan","id":3182036,"avatar_url":"https://avatars.githubusercontent.com/u/3182036?v=3","gravatar_id":"","url":"https://api.github.com/users/cloud-fan","html_url":"https://github.com/cloud-fan","followers_url":"https://api.github.com/users/cloud-fan/followers","following_url":"https://api.github.com/users/cloud-fan/following{/other_user}","gists_url":"https://api.github.com/users/cloud-fan/gists{/gist_id}","starred_url":"https://api.github.com/users/cloud-fan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cloud-fan/subscriptions","organizations_url":"https://api.github.com/users/cloud-fan/orgs","repos_url":"https://api.github.com/users/cloud-fan/repos","events_url":"https://api.github.com/users/cloud-fan/events{/privacy}","received_events_url":"https://api.github.com/users/cloud-fan/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":14,"created_at":"2016-11-07T16:04:56Z","updated_at":"2016-11-09T02:24:40Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/15797","html_url":"https://github.com/apache/spark/pull/15797","diff_url":"https://github.com/apache/spark/pull/15797.diff","patch_url":"https://github.com/apache/spark/pull/15797.patch"},"body":"## What changes were proposed in this pull request?\r\n\r\nThis PR corrects several partition related behaviors of `ExternalCatalog`:\r\n\r\n1. default partition location should not always lower case the partition column names in path string(fix `HiveExternalCatalog`)\r\n2. rename partition should not always lower case the partition column names in updated partition path string(fix `HiveExternalCatalog`)\r\n3. rename partition should update the partition location only for managed table(fix `InMemoryCatalog`)\r\n4. create partition with existing directory should be fine(fix `InMemoryCatalog`)\r\n5. create partition with non-existing directory should create that directory(fix `InMemoryCatalog`)\r\n6. drop partition from external table should not delete the directory(fix `InMemoryCatalog`)\r\n\r\n## How was this patch tested?\r\n\r\nnew tests in `ExternalCatalogSuite`\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/259319400","html_url":"https://github.com/apache/spark/pull/15797#issuecomment-259319400","issue_url":"https://api.github.com/repos/apache/spark/issues/15797","id":259319400,"user":{"login":"AmplabJenkins","id":4031291,"avatar_url":"https://avatars.githubusercontent.com/u/4031291?v=3","gravatar_id":"","url":"https://api.github.com/users/AmplabJenkins","html_url":"https://github.com/AmplabJenkins","followers_url":"https://api.github.com/users/AmplabJenkins/followers","following_url":"https://api.github.com/users/AmplabJenkins/following{/other_user}","gists_url":"https://api.github.com/users/AmplabJenkins/gists{/gist_id}","starred_url":"https://api.github.com/users/AmplabJenkins/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/AmplabJenkins/subscriptions","organizations_url":"https://api.github.com/users/AmplabJenkins/orgs","repos_url":"https://api.github.com/users/AmplabJenkins/repos","events_url":"https://api.github.com/users/AmplabJenkins/events{/privacy}","received_events_url":"https://api.github.com/users/AmplabJenkins/received_events","type":"User","site_admin":false},"created_at":"2016-11-09T02:24:40Z","updated_at":"2016-11-09T02:24:40Z","body":"Merged build finished. Test FAILed."}}
{"actor":{"display_login":"AmplabJenkins"}}