Skip to content

Conversation

@vanzin
Copy link
Contributor

@vanzin vanzin commented Dec 31, 2015

A slight adjustment to the checker configuration was needed; there is
a handful of warnings still left, but those are because of a bug in
the checker that I'll fix separately (before enabling errors for the
checker, of course).

A slight adjustment to the checker configuration was needed; there is
a handful of warnings still left, but those are because of a bug in
the checker that I'll fix separately (before enabling errors for the
checker, of course).
@SparkQA
Copy link

SparkQA commented Dec 31, 2015

Test build #48527 has finished for PR 10535 at commit ec571ef.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@rxin
Copy link
Contributor

rxin commented Dec 31, 2015

LGTM

@SparkQA
Copy link

SparkQA commented Dec 31, 2015

Test build #2276 has finished for PR 10535 at commit ec571ef.

  • This patch passes all tests.
  • This patch does not merge cleanly.
  • This patch adds no public classes.

@rxin
Copy link
Contributor

rxin commented Jan 1, 2016

I've merged this.

@asfgit asfgit closed this in a59a357 Jan 1, 2016
@vanzin vanzin deleted the SPARK-3873-mllib branch January 6, 2016 01:00
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.

3 participants