Skip to content

Conversation

@hlin09
Copy link
Contributor

@hlin09 hlin09 commented Apr 13, 2015

  1. Only use cleanClosure in creation of RRDDs. Normally, user and developer do not need to call cleanClosure in their function definition.
  2. Removes redundant code (e.g. unnecessary wrapper functions) related to cleanClosure.

@concretevitamin
Copy link
Contributor

Jenkins, test this please.

@concretevitamin
Copy link
Contributor

Hey @pwendell I can't seem to summon Jenkins now :(

@shivaram
Copy link
Contributor

Jenkins, test this please

@SparkQA
Copy link

SparkQA commented Apr 13, 2015

Test build #30200 has finished for PR 5495 at commit 74ec303.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.
  • This patch does not change any dependencies.

@shivaram
Copy link
Contributor

LGTM. @concretevitamin did you have any comments ?

@concretevitamin
Copy link
Contributor

LGTM

@asfgit asfgit closed this in 0ba3fdd Apr 14, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants