Skip to content

Conversation

@mengxr
Copy link
Contributor

@mengxr mengxr commented Mar 25, 2015

Though I don't see any bug in the existing code, the update in this PR makes it read better. @rxin

@rxin
Copy link
Contributor

rxin commented Mar 25, 2015

LGTM.

@andrewor14
Copy link
Contributor

LGTM2

@AmplabJenkins
Copy link

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29125/
Test FAILed.

@andrewor14
Copy link
Contributor

retest this please

@SparkQA
Copy link

SparkQA commented Mar 25, 2015

Test build #29131 has started for PR 5176 at commit 134494d.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Mar 25, 2015

Test build #29131 has finished for PR 5176 at commit 134494d.

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

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29131/
Test PASSed.

@andrewor14
Copy link
Contributor

Master

@asfgit asfgit closed this in c14ddd9 Mar 25, 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.

5 participants