[GitHub] spark pull request: SPARK-4022 [CORE] [MLLIB] Replace colt depende...

2014-10-27 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/2928#issuecomment-60638201 LGTM. Verified that `commons.math3` is shaded in the assembly jar. Merged into master. Thanks! --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request: SPARK-4022 [CORE] [MLLIB] Replace colt depende...

2014-10-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/2928 --- 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: SPARK-4022 [CORE] [MLLIB] Replace colt depende...

2014-10-26 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/2928#issuecomment-60529854 @mengxr Oops I missed that this failed when I saw the SQL tests failed. Should be OK. I rebased too for good measure. --- If your project is set up for it, you can reply

[GitHub] spark pull request: SPARK-4022 [CORE] [MLLIB] Replace colt depende...

2014-10-26 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/2928#issuecomment-60529896 [Test build #22253 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22253/consoleFull) for PR 2928 at commit

[GitHub] spark pull request: SPARK-4022 [CORE] [MLLIB] Replace colt depende...

2014-10-26 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/2928#issuecomment-60532479 [Test build #22253 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22253/consoleFull) for PR 2928 at commit

[GitHub] spark pull request: SPARK-4022 [CORE] [MLLIB] Replace colt depende...

2014-10-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/2928#issuecomment-60532482 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: SPARK-4022 [CORE] [MLLIB] Replace colt depende...

2014-10-25 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/2928#issuecomment-60505131 test this please --- 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

[GitHub] spark pull request: SPARK-4022 [CORE] [MLLIB] Replace colt depende...

2014-10-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/2928#issuecomment-60505236 [Test build #9 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/9/consoleFull) for PR 2928 at commit

[GitHub] spark pull request: SPARK-4022 [CORE] [MLLIB] Replace colt depende...

2014-10-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/2928#issuecomment-60506015 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: SPARK-4022 [CORE] [MLLIB] Replace colt depende...

2014-10-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/2928#issuecomment-60506012 [Test build #9 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/9/consoleFull) for PR 2928 at commit

[GitHub] spark pull request: SPARK-4022 [CORE] [MLLIB] Replace colt depende...

2014-10-25 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/2928#issuecomment-60506206 @srowen Could you check `JavaAPISuite.sample`? We need to update that test as well. --- If your project is set up for it, you can reply to this email and have your reply