Skip to content

[SPARK-12327][SPARKR] fix code for lintr warning for commented code - #10408

Closed
felixcheung wants to merge 1 commit into
apache:masterfrom
felixcheung:rcodecomment
Closed

[SPARK-12327][SPARKR] fix code for lintr warning for commented code#10408
felixcheung wants to merge 1 commit into
apache:masterfrom
felixcheung:rcodecomment

Conversation

@felixcheung

@felixcheungfelixcheung commented Dec 21, 2015

Copy link
Copy Markdown
Member

@SparkQA

SparkQA commented Dec 21, 2015

Copy link
Copy Markdown

Test build #48092 has finished for PR 10408 at commit a4f47a2.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@felixcheung

felixcheung commented Dec 29, 2015

Copy link
Copy Markdown
MemberAuthor

@shivaram ?

@shivaram

shivaram commented Jan 3, 2016

Copy link
Copy Markdown
Contributor

LGTM. Thanks @felixcheung -- Merging this to master and branch-1.6

asfgit pushed a commit that referenced this pull request Jan 3, 2016
@felixcheung@shivaram
[SPARK-12327][SPARKR] fix code for lintr warning for commented code
4e9dd16
shivaram
Author: felixcheung <felixcheung_m@hotmail.com>
Closes#10408 from felixcheung/rcodecomment.
(cherry picked from commit c3d5056)
Signed-off-by: Shivaram Venkataraman <shivaram@cs.berkeley.edu>
@asfgitasfgit closed this in c3d5056Jan 3, 2016
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@felixcheung@SparkQA@shivaram