{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/15690","repository_url":"https://api.github.com/repos/apache/spark","labels_url":"https://api.github.com/repos/apache/spark/issues/15690/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/15690/comments","events_url":"https://api.github.com/repos/apache/spark/issues/15690/events","html_url":"https://github.com/apache/spark/pull/15690","id":186156296,"number":15690,"title":"[SPARK-18174] [SQL] Avoid Implicit Type Cast in Arguments of Expressions Extending String2StringExpression","user":{"login":"gatorsmile","id":11567269,"avatar_url":"https://avatars.githubusercontent.com/u/11567269?v=3","gravatar_id":"","url":"https://api.github.com/users/gatorsmile","html_url":"https://github.com/gatorsmile","followers_url":"https://api.github.com/users/gatorsmile/followers","following_url":"https://api.github.com/users/gatorsmile/following{/other_user}","gists_url":"https://api.github.com/users/gatorsmile/gists{/gist_id}","starred_url":"https://api.github.com/users/gatorsmile/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gatorsmile/subscriptions","organizations_url":"https://api.github.com/users/gatorsmile/orgs","repos_url":"https://api.github.com/users/gatorsmile/repos","events_url":"https://api.github.com/users/gatorsmile/events{/privacy}","received_events_url":"https://api.github.com/users/gatorsmile/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2016-10-30T19:17:57Z","updated_at":"2016-10-30T21:33:43Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/15690","html_url":"https://github.com/apache/spark/pull/15690","diff_url":"https://github.com/apache/spark/pull/15690.diff","patch_url":"https://github.com/apache/spark/pull/15690.patch"},"body":"### What changes were proposed in this pull request?\r\nFor the expressions that extend `String2StringExpression` (including, `lower`, `upper`, `ltrim`, `rtrim`, `trim` and `reverse`), `Analyzer` should not implicitly cast the arguments to string. Instead, if users input the some data types instead of string, we should issue an exception for this misuse.\r\n\r\n### How was this patch tested?\r\nAdded a test case in the `StringFunctionsSuite`"},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/257180958","html_url":"https://github.com/apache/spark/pull/15690#issuecomment-257180958","issue_url":"https://api.github.com/repos/apache/spark/issues/15690","id":257180958,"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-10-30T21:33:42Z","updated_at":"2016-10-30T21:33:42Z","body":"Test PASSed.\nRefer to this link for build results (access rights to CI server needed): \nhttps://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67791/\nTest PASSed."}}
{"actor":{"display_login":"AmplabJenkins"}}