[GitHub] spark pull request: [SPARK-3873] [build] Enable import ordering er...

2016-01-10 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/10612 --- 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: [SPARK-3873] [build] Enable import ordering er...

2016-01-10 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-170426738 I've merged this. Thanks. --- 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 th

[GitHub] spark pull request: [SPARK-3873] [build] Enable import ordering er...

2016-01-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-170426634 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: [SPARK-3873] [build] Enable import ordering er...

2016-01-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-170426632 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: [SPARK-3873] [build] Enable import ordering er...

2016-01-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-170426571 **[Test build #49081 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49081/consoleFull)** for PR 10612 at commit [`2226c9d`](https://g

[GitHub] spark pull request: [SPARK-3873] [build] Enable import ordering er...

2016-01-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-170413869 **[Test build #49081 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49081/consoleFull)** for PR 10612 at commit [`2226c9d`](https://gi

[GitHub] spark pull request: [SPARK-3873] [build] Enable import ordering er...

2016-01-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-170411506 **[Test build #49077 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49077/consoleFull)** for PR 10612 at commit [`782f6de`](https://g

[GitHub] spark pull request: [SPARK-3873] [build] Enable import ordering er...

2016-01-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-170411509 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: [SPARK-3873] [build] Enable import ordering er...

2016-01-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-170411508 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: [SPARK-3873] [build] Enable import ordering er...

2016-01-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-170411305 **[Test build #49077 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49077/consoleFull)** for PR 10612 at commit [`782f6de`](https://gi

[GitHub] spark pull request: [SPARK-3873] [build] Enable import ordering er...

2016-01-06 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-169575497 You sh --- 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 a

[GitHub] spark pull request: [SPARK-3873] [build] Enable import ordering er...

2016-01-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-169547938 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: [SPARK-3873] [build] Enable import ordering er...

2016-01-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-169547941 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: [SPARK-3873] [build] Enable import ordering er...

2016-01-06 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-169547519 **[Test build #4 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/4/consoleFull)** for PR 10612 at commit [`6da306d`](https://g

[GitHub] spark pull request: [SPARK-3873] [build] Enable import ordering er...

2016-01-06 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-169525469 **[Test build #4 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/4/consoleFull)** for PR 10612 at commit [`6da306d`](https://gi

[GitHub] spark pull request: [SPARK-3873] [build] Enable import ordering er...

2016-01-06 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/10612#discussion_r49032989 --- Diff: scalastyle-config.xml --- @@ -187,6 +187,17 @@ This file is divided into 3 sections: scala.collection.JavaConverters._ and use .asScala / .asJ

[GitHub] spark pull request: [SPARK-3873] [build] Enable import ordering er...

2016-01-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-169448630 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: [SPARK-3873] [build] Enable import ordering er...

2016-01-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-169448628 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: [SPARK-3873] [build] Enable import ordering er...

2016-01-06 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-169448100 **[Test build #48862 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48862/consoleFull)** for PR 10612 at commit [`35569f9`](https://g

[GitHub] spark pull request: [SPARK-3873] [build] Enable import ordering er...

2016-01-06 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-169404809 **[Test build #48862 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48862/consoleFull)** for PR 10612 at commit [`35569f9`](https://gi

[GitHub] spark pull request: [SPARK-3873] [build] Enable import ordering er...

2016-01-05 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-169236875 **[Test build #48834 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48834/consoleFull)** for PR 10612 at commit [`930ccd3`](https://g

[GitHub] spark pull request: [SPARK-3873] [build] Enable import ordering er...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-169236888 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: [SPARK-3873] [build] Enable import ordering er...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-169236887 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: [SPARK-3873] [build] Enable import ordering er...

2016-01-05 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-169236187 **[Test build #48834 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48834/consoleFull)** for PR 10612 at commit [`930ccd3`](https://gi

[GitHub] spark pull request: [SPARK-3873] [build] Enable import ordering er...

2016-01-05 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-169233964 retest 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 fea

[GitHub] spark pull request: [SPARK-3873] [build] Enable import ordering er...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-169216596 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: [SPARK-3873] [build] Enable import ordering er...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-169216595 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: [SPARK-3873] [build] Enable import ordering er...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-169210571 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: [SPARK-3873] [build] Enable import ordering er...

2016-01-05 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-169210567 **[Test build #48816 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48816/consoleFull)** for PR 10612 at commit [`1f08d6e`](https://g

[GitHub] spark pull request: [SPARK-3873] [build] Enable import ordering er...

2016-01-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-169210572 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: [SPARK-3873] [build] Enable import ordering er...

2016-01-05 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-169209019 **[Test build #48816 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48816/consoleFull)** for PR 10612 at commit [`1f08d6e`](https://gi

[GitHub] spark pull request: [SPARK-3873] [build] Enable import ordering er...

2016-01-05 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/10612#issuecomment-169207716 I'll let this simmer for a few days to wait for new PRs that might have violations to go in, and fix those... but at some point I'll get bored of fixing things and proba

[GitHub] spark pull request: [SPARK-3873] [build] Enable import ordering er...

2016-01-05 Thread vanzin
GitHub user vanzin opened a pull request: https://github.com/apache/spark/pull/10612 [SPARK-3873] [build] Enable import ordering error checking. Turn import ordering violations into build errors, plus a few adjustments to account for how the checker behaves. I'm a little on the f