Skip to content

Conversation

@rxin
Copy link
Contributor

@rxin rxin commented Sep 29, 2014

@SparkQA
Copy link

SparkQA commented Sep 29, 2014

QA tests have started for PR 2580 at commit 323dfec.

  • This patch merges cleanly.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two less spaces before '}'

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No the indent was correct.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry I looked the wrong line.

@SparkQA
Copy link

SparkQA commented Sep 29, 2014

QA tests have started for PR 2580 at commit 5814292.

  • This patch merges cleanly.

@aarondav
Copy link
Contributor

LGTM once Travis passes.

@SparkQA
Copy link

SparkQA commented Sep 29, 2014

QA tests have finished for PR 2580 at commit 323dfec.

  • This patch passes unit 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/20982/

@aarondav
Copy link
Contributor

Merging into master.

@asfgit asfgit closed this in e43c72f Sep 29, 2014
@SparkQA
Copy link

SparkQA commented Sep 29, 2014

QA tests have finished for PR 2580 at commit 5814292.

  • This patch passes unit 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/20985/

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