{"action":"created","comment":{"url":"https://api.github.com/repos/apache/spark/pulls/comments/47612640","id":47612640,"diff_hunk":"@@ -1357,17 +1357,46 @@ setMethod(\"mutate\",\n           function(.data, ...) {\n             x <- .data\n             cols <- list(...)\n-            stopifnot(length(cols) > 0)\n+            if (length(cols) <= 0) {\n+              return(x)\n+            }\n+\n             stopifnot(class(cols[[1]]) == \"Column\")\n+\n+            # Check if there is any duplicated column name in the DataFrame\n+            dfCols <- columns(x)\n+            if (length(unique(dfCols)) != length(dfCols)) {","path":"R/pkg/R/DataFrame.R","position":27,"original_position":27,"commit_id":"ac34da1e8385161fc50ce21a0595437a0854d78b","original_commit_id":"ac34da1e8385161fc50ce21a0595437a0854d78b","user":{"login":"3ourroom","id":10443582,"avatar_url":"https://avatars.githubusercontent.com/u/10443582?v=3","gravatar_id":"","url":"https://api.github.com/users/3ourroom","html_url":"https://github.com/3ourroom","followers_url":"https://api.github.com/users/3ourroom/followers","following_url":"https://api.github.com/users/3ourroom/following{/other_user}","gists_url":"https://api.github.com/users/3ourroom/gists{/gist_id}","starred_url":"https://api.github.com/users/3ourroom/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/3ourroom/subscriptions","organizations_url":"https://api.github.com/users/3ourroom/orgs","repos_url":"https://api.github.com/users/3ourroom/repos","events_url":"https://api.github.com/users/3ourroom/events{/privacy}","received_events_url":"https://api.github.com/users/3ourroom/received_events","type":"User","site_admin":false},"body":"\nNAVER - http://www.naver.com/\n--------------------------------------------\n\n3ourroom@naver.com 님께 보내신 메일 <Re: [spark] [SPARK-12235][SPARKR] Enhance mutate() to support replace existing columns. (#10220)> 이 다음과 같은 이유로 전송 실패했습니다.\n\n--------------------------------------------\n\n받는 사람이 회원님의 메일을 수신차단 하였습니다. \n\n\n--------------------------------------------\n","created_at":"2015-12-15T09:16:27Z","updated_at":"2015-12-15T09:16:27Z","html_url":"https://github.com/apache/spark/pull/10220#discussion_r47612640","pull_request_url":"https://api.github.com/repos/apache/spark/pulls/10220","_links":{"self":{"href":"https://api.github.com/repos/apache/spark/pulls/comments/47612640"},"html":{"href":"https://github.com/apache/spark/pull/10220#discussion_r47612640"},"pull_request":{"href":"https://api.github.com/repos/apache/spark/pulls/10220"}}},"pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/10220","id":53084473,"html_url":"https://github.com/apache/spark/pull/10220","diff_url":"https://github.com/apache/spark/pull/10220.diff","patch_url":"https://github.com/apache/spark/pull/10220.patch","issue_url":"https://api.github.com/repos/apache/spark/issues/10220","number":10220,"state":"open","locked":false,"title":"[SPARK-12235][SPARKR] Enhance mutate() to support replace existing columns.","user":{"login":"sun-rui","id":2528327,"avatar_url":"https://avatars.githubusercontent.com/u/2528327?v=3","gravatar_id":"","url":"https://api.github.com/users/sun-rui","html_url":"https://github.com/sun-rui","followers_url":"https://api.github.com/users/sun-rui/followers","following_url":"https://api.github.com/users/sun-rui/following{/other_user}","gists_url":"https://api.github.com/users/sun-rui/gists{/gist_id}","starred_url":"https://api.github.com/users/sun-rui/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sun-rui/subscriptions","organizations_url":"https://api.github.com/users/sun-rui/orgs","repos_url":"https://api.github.com/users/sun-rui/repos","events_url":"https://api.github.com/users/sun-rui/events{/privacy}","received_events_url":"https://api.github.com/users/sun-rui/received_events","type":"User","site_admin":false},"body":"Make the behavior of mutate more consistent with that in dplyr, besides support for replacing existing columns.\r\n1. Throw error message when there are duplicated column names in the DataFrame being mutated.\r\n2. when there are duplicated column names in specified columns by arguments, the last column of the same name takes effect.","created_at":"2015-12-09T07:15:28Z","updated_at":"2015-12-15T09:16:27Z","closed_at":null,"merged_at":null,"merge_commit_sha":"cc70b97acdc2c9ec8dafbe54f32b829a7a40cf3d","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/apache/spark/pulls/10220/commits","review_comments_url":"https://api.github.com/repos/apache/spark/pulls/10220/comments","review_comment_url":"https://api.github.com/repos/apache/spark/pulls/comments{/number}","comments_url":"https://api.github.com/repos/apache/spark/issues/10220/comments","statuses_url":"https://api.github.com/repos/apache/spark/statuses/ac34da1e8385161fc50ce21a0595437a0854d78b","head":{"label":"sun-rui:SPARK-12235","ref":"SPARK-12235","sha":"ac34da1e8385161fc50ce21a0595437a0854d78b","user":{"login":"sun-rui","id":2528327,"avatar_url":"https://avatars.githubusercontent.com/u/2528327?v=3","gravatar_id":"","url":"https://api.github.com/users/sun-rui","html_url":"https://github.com/sun-rui","followers_url":"https://api.github.com/users/sun-rui/followers","following_url":"https://api.github.com/users/sun-rui/following{/other_user}","gists_url":"https://api.github.com/users/sun-rui/gists{/gist_id}","starred_url":"https://api.github.com/users/sun-rui/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sun-rui/subscriptions","organizations_url":"https://api.github.com/users/sun-rui/orgs","repos_url":"https://api.github.com/users/sun-rui/repos","events_url":"https://api.github.com/users/sun-rui/events{/privacy}","received_events_url":"https://api.github.com/users/sun-rui/received_events","type":"User","site_admin":false},"repo":{"id":34327558,"name":"spark","full_name":"sun-rui/spark","owner":{"login":"sun-rui","id":2528327,"avatar_url":"https://avatars.githubusercontent.com/u/2528327?v=3","gravatar_id":"","url":"https://api.github.com/users/sun-rui","html_url":"https://github.com/sun-rui","followers_url":"https://api.github.com/users/sun-rui/followers","following_url":"https://api.github.com/users/sun-rui/following{/other_user}","gists_url":"https://api.github.com/users/sun-rui/gists{/gist_id}","starred_url":"https://api.github.com/users/sun-rui/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sun-rui/subscriptions","organizations_url":"https://api.github.com/users/sun-rui/orgs","repos_url":"https://api.github.com/users/sun-rui/repos","events_url":"https://api.github.com/users/sun-rui/events{/privacy}","received_events_url":"https://api.github.com/users/sun-rui/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/sun-rui/spark","description":"Mirror of Apache Spark","fork":true,"url":"https://api.github.com/repos/sun-rui/spark","forks_url":"https://api.github.com/repos/sun-rui/spark/forks","keys_url":"https://api.github.com/repos/sun-rui/spark/keys{/key_id}","collaborators_url":"https://api.github.com/repos/sun-rui/spark/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/sun-rui/spark/teams","hooks_url":"https://api.github.com/repos/sun-rui/spark/hooks","issue_events_url":"https://api.github.com/repos/sun-rui/spark/issues/events{/number}","events_url":"https://api.github.com/repos/sun-rui/spark/events","assignees_url":"https://api.github.com/repos/sun-rui/spark/assignees{/user}","branches_url":"https://api.github.com/repos/sun-rui/spark/branches{/branch}","tags_url":"https://api.github.com/repos/sun-rui/spark/tags","blobs_url":"https://api.github.com/repos/sun-rui/spark/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/sun-rui/spark/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/sun-rui/spark/git/refs{/sha}","trees_url":"https://api.github.com/repos/sun-rui/spark/git/trees{/sha}","statuses_url":"https://api.github.com/repos/sun-rui/spark/statuses/{sha}","languages_url":"https://api.github.com/repos/sun-rui/spark/languages","stargazers_url":"https://api.github.com/repos/sun-rui/spark/stargazers","contributors_url":"https://api.github.com/repos/sun-rui/spark/contributors","subscribers_url":"https://api.github.com/repos/sun-rui/spark/subscribers","subscription_url":"https://api.github.com/repos/sun-rui/spark/subscription","commits_url":"https://api.github.com/repos/sun-rui/spark/commits{/sha}","git_commits_url":"https://api.github.com/repos/sun-rui/spark/git/commits{/sha}","comments_url":"https://api.github.com/repos/sun-rui/spark/comments{/number}","issue_comment_url":"https://api.github.com/repos/sun-rui/spark/issues/comments{/number}","contents_url":"https://api.github.com/repos/sun-rui/spark/contents/{+path}","compare_url":"https://api.github.com/repos/sun-rui/spark/compare/{base}...{head}","merges_url":"https://api.github.com/repos/sun-rui/spark/merges","archive_url":"https://api.github.com/repos/sun-rui/spark/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/sun-rui/spark/downloads","issues_url":"https://api.github.com/repos/sun-rui/spark/issues{/number}","pulls_url":"https://api.github.com/repos/sun-rui/spark/pulls{/number}","milestones_url":"https://api.github.com/repos/sun-rui/spark/milestones{/number}","notifications_url":"https://api.github.com/repos/sun-rui/spark/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/sun-rui/spark/labels{/name}","releases_url":"https://api.github.com/repos/sun-rui/spark/releases{/id}","created_at":"2015-04-21T13:08:14Z","updated_at":"2015-04-21T13:08:34Z","pushed_at":"2015-12-15T06:43:12Z","git_url":"git://github.com/sun-rui/spark.git","ssh_url":"git@github.com:sun-rui/spark.git","clone_url":"https://github.com/sun-rui/spark.git","svn_url":"https://github.com/sun-rui/spark","homepage":null,"size":133119,"stargazers_count":0,"watchers_count":0,"language":"Scala","has_issues":false,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"apache:master","ref":"master","sha":"3934562d34bbe08d91c54b4bbee27870e93d7571","user":{"login":"apache","id":47359,"avatar_url":"https://avatars.githubusercontent.com/u/47359?v=3","gravatar_id":"","url":"https://api.github.com/users/apache","html_url":"https://github.com/apache","followers_url":"https://api.github.com/users/apache/followers","following_url":"https://api.github.com/users/apache/following{/other_user}","gists_url":"https://api.github.com/users/apache/gists{/gist_id}","starred_url":"https://api.github.com/users/apache/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/apache/subscriptions","organizations_url":"https://api.github.com/users/apache/orgs","repos_url":"https://api.github.com/users/apache/repos","events_url":"https://api.github.com/users/apache/events{/privacy}","received_events_url":"https://api.github.com/users/apache/received_events","type":"Organization","site_admin":false},"repo":{"id":17165658,"name":"spark","full_name":"apache/spark","owner":{"login":"apache","id":47359,"avatar_url":"https://avatars.githubusercontent.com/u/47359?v=3","gravatar_id":"","url":"https://api.github.com/users/apache","html_url":"https://github.com/apache","followers_url":"https://api.github.com/users/apache/followers","following_url":"https://api.github.com/users/apache/following{/other_user}","gists_url":"https://api.github.com/users/apache/gists{/gist_id}","starred_url":"https://api.github.com/users/apache/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/apache/subscriptions","organizations_url":"https://api.github.com/users/apache/orgs","repos_url":"https://api.github.com/users/apache/repos","events_url":"https://api.github.com/users/apache/events{/privacy}","received_events_url":"https://api.github.com/users/apache/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/apache/spark","description":"Mirror of Apache Spark","fork":false,"url":"https://api.github.com/repos/apache/spark","forks_url":"https://api.github.com/repos/apache/spark/forks","keys_url":"https://api.github.com/repos/apache/spark/keys{/key_id}","collaborators_url":"https://api.github.com/repos/apache/spark/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/apache/spark/teams","hooks_url":"https://api.github.com/repos/apache/spark/hooks","issue_events_url":"https://api.github.com/repos/apache/spark/issues/events{/number}","events_url":"https://api.github.com/repos/apache/spark/events","assignees_url":"https://api.github.com/repos/apache/spark/assignees{/user}","branches_url":"https://api.github.com/repos/apache/spark/branches{/branch}","tags_url":"https://api.github.com/repos/apache/spark/tags","blobs_url":"https://api.github.com/repos/apache/spark/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/apache/spark/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/apache/spark/git/refs{/sha}","trees_url":"https://api.github.com/repos/apache/spark/git/trees{/sha}","statuses_url":"https://api.github.com/repos/apache/spark/statuses/{sha}","languages_url":"https://api.github.com/repos/apache/spark/languages","stargazers_url":"https://api.github.com/repos/apache/spark/stargazers","contributors_url":"https://api.github.com/repos/apache/spark/contributors","subscribers_url":"https://api.github.com/repos/apache/spark/subscribers","subscription_url":"https://api.github.com/repos/apache/spark/subscription","commits_url":"https://api.github.com/repos/apache/spark/commits{/sha}","git_commits_url":"https://api.github.com/repos/apache/spark/git/commits{/sha}","comments_url":"https://api.github.com/repos/apache/spark/comments{/number}","issue_comment_url":"https://api.github.com/repos/apache/spark/issues/comments{/number}","contents_url":"https://api.github.com/repos/apache/spark/contents/{+path}","compare_url":"https://api.github.com/repos/apache/spark/compare/{base}...{head}","merges_url":"https://api.github.com/repos/apache/spark/merges","archive_url":"https://api.github.com/repos/apache/spark/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/apache/spark/downloads","issues_url":"https://api.github.com/repos/apache/spark/issues{/number}","pulls_url":"https://api.github.com/repos/apache/spark/pulls{/number}","milestones_url":"https://api.github.com/repos/apache/spark/milestones{/number}","notifications_url":"https://api.github.com/repos/apache/spark/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/apache/spark/labels{/name}","releases_url":"https://api.github.com/repos/apache/spark/releases{/id}","created_at":"2014-02-25T08:00:08Z","updated_at":"2015-12-15T07:50:58Z","pushed_at":"2015-12-15T07:50:17Z","git_url":"git://github.com/apache/spark.git","ssh_url":"git@github.com:apache/spark.git","clone_url":"https://github.com/apache/spark.git","svn_url":"https://github.com/apache/spark","homepage":null,"size":389879,"stargazers_count":6580,"watchers_count":6580,"language":"Scala","has_issues":false,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":6096,"mirror_url":"git://git.apache.org/spark.git","open_issues_count":411,"forks":6096,"open_issues":411,"watchers":6580,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/apache/spark/pulls/10220"},"html":{"href":"https://github.com/apache/spark/pull/10220"},"issue":{"href":"https://api.github.com/repos/apache/spark/issues/10220"},"comments":{"href":"https://api.github.com/repos/apache/spark/issues/10220/comments"},"review_comments":{"href":"https://api.github.com/repos/apache/spark/pulls/10220/comments"},"review_comment":{"href":"https://api.github.com/repos/apache/spark/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/apache/spark/pulls/10220/commits"},"statuses":{"href":"https://api.github.com/repos/apache/spark/statuses/ac34da1e8385161fc50ce21a0595437a0854d78b"}}}}