{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/13306","repository_url":"https://api.github.com/repos/apache/spark","labels_url":"https://api.github.com/repos/apache/spark/issues/13306/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/13306/comments","events_url":"https://api.github.com/repos/apache/spark/issues/13306/events","html_url":"https://github.com/apache/spark/pull/13306","id":156864132,"number":13306,"title":"[SPARK-12988][SQL]Can't drop top level columns that contain dots","user":{"login":"clockfly","id":2595532,"avatar_url":"https://avatars.githubusercontent.com/u/2595532?v=3","gravatar_id":"","url":"https://api.github.com/users/clockfly","html_url":"https://github.com/clockfly","followers_url":"https://api.github.com/users/clockfly/followers","following_url":"https://api.github.com/users/clockfly/following{/other_user}","gists_url":"https://api.github.com/users/clockfly/gists{/gist_id}","starred_url":"https://api.github.com/users/clockfly/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/clockfly/subscriptions","organizations_url":"https://api.github.com/users/clockfly/orgs","repos_url":"https://api.github.com/users/clockfly/repos","events_url":"https://api.github.com/users/clockfly/events{/privacy}","received_events_url":"https://api.github.com/users/clockfly/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2016-05-25T22:24:36Z","updated_at":"2016-05-26T00:12:16Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/13306","html_url":"https://github.com/apache/spark/pull/13306","diff_url":"https://github.com/apache/spark/pull/13306.diff","patch_url":"https://github.com/apache/spark/pull/13306.patch"},"body":"## What changes were proposed in this pull request?\r\n\r\nFixes \"Can't drop top level columns that contain dots\".\r\n\r\nThis work is based on @dilipbiswal's https://github.com/apache/spark/pull/10943. \r\nThis PR fixes problems like:\r\n\r\n```\r\nscala> Seq((1, 2)).toDF(\"a.b\", \"a.c\").drop(\"a.b\")\r\norg.apache.spark.sql.AnalysisException: cannot resolve '`a.c`' given input columns: [a.b, a.c];\r\n```\r\n\r\n`drop(columnName)` can only be used to drop top level column, so, we should parse the column name literally WITHOUT interpreting dot \".\"\r\n\r\nWe should also NOT interpret back tick \"`\", otherwise it is hard to understand what \r\n\r\n```\r\n​```aaa```bbb``\r\n```\r\n\r\nactually means. \r\n\r\n## How was this patch tested?\r\n\r\nUnit tests.\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/221744301","html_url":"https://github.com/apache/spark/pull/13306#issuecomment-221744301","issue_url":"https://api.github.com/repos/apache/spark/issues/13306","id":221744301,"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-05-26T00:12:16Z","updated_at":"2016-05-26T00:12:16Z","body":"Test PASSed.\nRefer to this link for build results (access rights to CI server needed): \nhttps://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/59304/\nTest PASSed."}}
{"actor":{"display_login":"AmplabJenkins"}}