[GitHub] spark pull request: [Minor][ML][MLLIB] Remove unused imports

2016-04-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12497#issuecomment-213676321 Build finished. Test PASSed. --- 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

[GitHub] spark pull request: [Minor][ML][MLLIB] Remove unused imports

2016-04-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12497#issuecomment-213676323 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [Minor][ML][MLLIB] Remove unused imports

2016-04-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12497#issuecomment-213676118 **[Test build #56777 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56777/consoleFull)** for PR 12497 at commit [`90e57c8`](https://g

[GitHub] spark pull request: [Minor][ML][MLLIB] Remove unused imports

2016-04-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12497 --- 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 ena

[GitHub] spark pull request: [Minor][ML][MLLIB] Remove unused imports

2016-04-22 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/12497#issuecomment-213673399 Thanks - merging in master. --- 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: [Minor][ML][MLLIB] Remove unused imports

2016-04-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12497#issuecomment-213673237 Merged build finished. Test PASSed. --- 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 projec

[GitHub] spark pull request: [Minor][ML][MLLIB] Remove unused imports

2016-04-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12497#issuecomment-213673239 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [Minor][ML][MLLIB] Remove unused imports

2016-04-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12497#issuecomment-213673202 **[Test build #56778 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56778/consoleFull)** for PR 12497 at commit [`ab42268`](https://g

[GitHub] spark pull request: [Minor][ML][MLLIB] Remove unused imports

2016-04-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12497#issuecomment-213663512 **[Test build #56778 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56778/consoleFull)** for PR 12497 at commit [`ab42268`](https://gi

[GitHub] spark pull request: [Minor][ML][MLLIB] Remove unused imports

2016-04-22 Thread zhengruifeng
Github user zhengruifeng commented on the pull request: https://github.com/apache/spark/pull/12497#issuecomment-213663449 @srowen I have reviewed all scala files in Graphx and some in SQL. And remove another some unused imports in this PR. --- If your project is set up for it, you ca

[GitHub] spark pull request: [Minor][ML][MLLIB] Remove unused imports

2016-04-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12497#issuecomment-213663339 **[Test build #56777 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56777/consoleFull)** for PR 12497 at commit [`90e57c8`](https://gi

[GitHub] spark pull request: [Minor][ML][MLLIB] Remove unused imports

2016-04-22 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/12497#issuecomment-213437912 @zhengruifeng this one needs a rebase, and you can remove more unused imports per above while you're at it --- If your project is set up for it, you can reply to this e

[GitHub] spark pull request: [Minor][ML][MLLIB] Remove unused imports

2016-04-20 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/12497#issuecomment-212366396 Agree, of course we have to test the result of removing "unused" imports in any event, and that's less obvious for Scala. It gets it right most of the time. In any event

[GitHub] spark pull request: [Minor][ML][MLLIB] Remove unused imports

2016-04-20 Thread zhengruifeng
Github user zhengruifeng commented on the pull request: https://github.com/apache/spark/pull/12497#issuecomment-212364172 @srowen IDEA's unused hint may be wrong in some place. In ParamGridBuilder.scala, IDEA flags 'scala.annotation.varargs' as unused, which is actually needed in b

[GitHub] spark pull request: [Minor][ML][MLLIB] Remove unused imports

2016-04-20 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/12497#issuecomment-212361373 Since this is something that would be good to do globally, if at all, and there are automated tools to find this type of problem, I'd run an IDE over this to find and fi

[GitHub] spark pull request: [Minor][ML][MLLIB] Remove unused imports

2016-04-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12497#issuecomment-212184907 Merged build finished. Test PASSed. --- 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 projec

[GitHub] spark pull request: [Minor][ML][MLLIB] Remove unused imports

2016-04-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12497#issuecomment-212184814 **[Test build #56295 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56295/consoleFull)** for PR 12497 at commit [`8aff210`](https://g

[GitHub] spark pull request: [Minor][ML][MLLIB] Remove unused imports

2016-04-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12497#issuecomment-212184909 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [Minor][ML][MLLIB] Remove unused imports

2016-04-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12497#issuecomment-212178767 **[Test build #56295 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56295/consoleFull)** for PR 12497 at commit [`8aff210`](https://gi

[GitHub] spark pull request: [Minor][ML][MLLIB] Remove unused imports

2016-04-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12497#issuecomment-212177059 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [Minor][ML][MLLIB] Remove unused imports

2016-04-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12497#issuecomment-212177052 Merged build finished. Test FAILed. --- 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 projec

[GitHub] spark pull request: [Minor][ML][MLLIB] Remove unused imports

2016-04-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12497#issuecomment-212177027 **[Test build #56294 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56294/consoleFull)** for PR 12497 at commit [`b226bc6`](https://g

[GitHub] spark pull request: [Minor][ML][MLLIB] Remove unused imports

2016-04-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12497#issuecomment-212175806 **[Test build #56294 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56294/consoleFull)** for PR 12497 at commit [`b226bc6`](https://gi