{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/14518","repository_url":"https://api.github.com/repos/apache/spark","labels_url":"https://api.github.com/repos/apache/spark/issues/14518/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/14518/comments","events_url":"https://api.github.com/repos/apache/spark/issues/14518/events","html_url":"https://github.com/apache/spark/pull/14518","id":169724493,"number":14518,"title":"[SPARK-16610][SQL] Add `orc.compress` as an alias for `compression` option.","user":{"login":"HyukjinKwon","id":6477701,"avatar_url":"https://avatars.githubusercontent.com/u/6477701?v=3","gravatar_id":"","url":"https://api.github.com/users/HyukjinKwon","html_url":"https://github.com/HyukjinKwon","followers_url":"https://api.github.com/users/HyukjinKwon/followers","following_url":"https://api.github.com/users/HyukjinKwon/following{/other_user}","gists_url":"https://api.github.com/users/HyukjinKwon/gists{/gist_id}","starred_url":"https://api.github.com/users/HyukjinKwon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/HyukjinKwon/subscriptions","organizations_url":"https://api.github.com/users/HyukjinKwon/orgs","repos_url":"https://api.github.com/users/HyukjinKwon/repos","events_url":"https://api.github.com/users/HyukjinKwon/events{/privacy}","received_events_url":"https://api.github.com/users/HyukjinKwon/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2016-08-06T02:14:21Z","updated_at":"2016-08-08T02:12:18Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/14518","html_url":"https://github.com/apache/spark/pull/14518","diff_url":"https://github.com/apache/spark/pull/14518.diff","patch_url":"https://github.com/apache/spark/pull/14518.patch"},"body":"## What changes were proposed in this pull request?\r\n\r\nFor ORC source, Spark SQL has a writer option `compression`, which is used to set the codec and its value will be also set to `orc.compress` (the orc conf used for codec). However, if a user only set `orc.compress` in the writer option, we should not use the default value of `compression` (snappy) as the codec. Instead, we should respect the value of `orc.compress`.\r\n\r\nThis PR makes ORC data source not ignoring `orc.compress` when `comperssion` is unset.\r\n\r\nSo, here is the behaviour,\r\n\r\n 1. Check `compression` and use this if it is set.\r\n 2. If `compression` is not set, check `orc.compress` and use it.\r\n 3. If `compression` and `orc.compress` are not set, then use the default snappy.\r\n\r\n## How was this patch tested?\r\n\r\nUnit test in `OrcQuerySuite`.\r\n"},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/238127031","html_url":"https://github.com/apache/spark/pull/14518#issuecomment-238127031","issue_url":"https://api.github.com/repos/apache/spark/issues/14518","id":238127031,"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-08-08T02:12:18Z","updated_at":"2016-08-08T02:12:18Z","body":"Merged build finished. Test PASSed."}}
{"actor":{"display_login":"AmplabJenkins"}}