[GitHub] spark pull request: Fix some tests.

Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/917#issuecomment-46740805 Jenkins, test this please. Thanks for this - I wanted to get a build matrix setup in jenkins before looking at fixes for tests. Indeed, our tests do not work currently

[GitHub] spark pull request: Fix some tests.

Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/917#issuecomment-46740913 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] spark pull request: Fix some tests.

Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/917#issuecomment-46740912 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: Fix some tests.

Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/917#issuecomment-46741892 Merged build finished. All automated tests passed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] spark pull request: Fix some tests.

Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/917#issuecomment-46741895 All automated tests passed. Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/15974/ --- If your project

[GitHub] spark pull request: Fix some tests.

Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/917 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] spark pull request: Fix some tests.

Github user srowen commented on the pull request: https://github.com/apache/spark/pull/917#issuecomment-46526650 (Looks like this needs a rebase?) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: Fix some tests.

Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/917#issuecomment-46467351 Another friendly ping. Could I get some eyes on this change? It's pretty trivial. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: Fix some tests.

Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/917#issuecomment-45380499 Ping. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] spark pull request: Fix some tests.

Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/917#issuecomment-44710024 BTW I have a different version of the fix that changes RandomSamplerSuite / CacheManagerSuite to use mockito instead of easymock. Mockito doesn't seem to suffer from the

[GitHub] spark pull request: Fix some tests.

GitHub user vanzin opened a pull request: https://github.com/apache/spark/pull/917 Fix some tests. - JavaAPISuite was trying to compare a bare path with a URI. Fix by extracting the path from the URI, since we know it should be a local path anyway/ - b9be1609

[GitHub] spark pull request: Fix some tests.

Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/917#issuecomment-44607442 Can one of the admins verify this patch? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your