Skip to content

Conversation

@rxin
Copy link
Contributor

@rxin rxin commented Apr 13, 2015

Turn scalastyle on for all test code. Most of the violations have been resolved in my previous pull requests:

Core: #5484
SQL: #5412
MLlib: #5411
GraphX: #5410
Streaming: #5409

@SparkQA
Copy link

SparkQA commented Apr 13, 2015

Test build #30139 has finished for PR 5486 at commit 5060e6b.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • s" /mesos-home/bin/spark-class $
    • s"cd test-app-1*; ./bin/spark-class $
  • This patch does not change any dependencies.

@rxin
Copy link
Contributor Author

rxin commented Apr 13, 2015

Jenkins, test this please.

@SparkQA
Copy link

SparkQA commented Apr 13, 2015

Test build #30146 has finished for PR 5486 at commit 22014fc.

  • This patch fails Scala style tests.
  • This patch merges cleanly.
  • This patch adds no public classes.
  • This patch does not change any dependencies.

@rxin rxin force-pushed the test-style-enable branch from 22014fc to 01683de Compare April 13, 2015 06:08
@rxin
Copy link
Contributor Author

rxin commented Apr 13, 2015

Jenkins, test this please.

@SparkQA
Copy link

SparkQA commented Apr 13, 2015

Test build #30148 has finished for PR 5486 at commit 01683de.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.
  • This patch does not change any dependencies.

@rxin
Copy link
Contributor Author

rxin commented Apr 13, 2015

Jenkins, retest this please.

@SparkQA
Copy link

SparkQA commented Apr 13, 2015

Test build #30149 has finished for PR 5486 at commit 01683de.

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

@SparkQA
Copy link

SparkQA commented Apr 13, 2015

Test build #30154 has finished for PR 5486 at commit 01683de.

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

@asfgit asfgit closed this in c5b0b29 Apr 13, 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.

2 participants