Skip to content

Conversation

@y-shimizu
Copy link
Contributor

looks unnecessary 😀

@srowen
Copy link
Member

srowen commented Jan 29, 2015

Although it's unnecessary, I think this kind of thing is generally too trivial to bother opening a PR for by itself.

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@mengxr
Copy link
Contributor

mengxr commented Jan 29, 2015

Agree. But since we have a PR already, let's call Jenkins:)

ok to test

@mengxr
Copy link
Contributor

mengxr commented Jan 29, 2015

test this please

@SparkQA
Copy link

SparkQA commented Jan 29, 2015

Test build #26333 has started for PR 4268 at commit 12be0e9.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Jan 29, 2015

Test build #26333 has finished for PR 4268 at commit 12be0e9.

  • 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/26333/
Test PASSed.

@asfgit asfgit closed this in 5338772 Jan 30, 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