[GitHub] spark pull request: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-30 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/11542#issuecomment-203554149 I would veto removing 2.10 support at this time. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark pull request: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-30 Thread markhamstra
Github user markhamstra commented on the pull request: https://github.com/apache/spark/pull/11542#issuecomment-203444519 While I don't see the "I think that it is premature to drop support for 2.10.x." from @marmbrus as particularly ambiguous, I also don't think that a definite

[GitHub] spark pull request: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-30 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/11542#issuecomment-203436651 Yeah there isn't a 'here is the decision' message from a decider. Some in favor and no veto is the end state. Java 7 seems pretty unanimous even so I think we can take

[GitHub] spark pull request: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-30 Thread markhamstra
Github user markhamstra commented on the pull request: https://github.com/apache/spark/pull/11542#issuecomment-203433869 @srowen I see the decisions whether to drop Java 7 or Scala 2.10 as more than just code changes. I'll agree that there was no objection to dropping Java 7, but

[GitHub] spark pull request: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-30 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/11542#issuecomment-203376206 @markhamstra the PMC doesn't decide these things. In extreme cases it could be put to a vote. Code changes all work on lazy consensus by default. I actually

[GitHub] spark pull request: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-29 Thread markhamstra
Github user markhamstra commented on the pull request: https://github.com/apache/spark/pull/11542#issuecomment-203228945 One small correction: I believe Mridul was in favor of dropping Scala 2.10 support, so the expressed opinions of PMC members were mixed. --- If your project is

[GitHub] spark pull request: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11542#issuecomment-203228465 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

[GitHub] spark pull request: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11542#issuecomment-203228466 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11542#issuecomment-203228341 **[Test build #54482 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54482/consoleFull)** for PR 11542 at commit

[GitHub] spark pull request: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-29 Thread markhamstra
Github user markhamstra commented on the pull request: https://github.com/apache/spark/pull/11542#issuecomment-203227472 Thanks. Before the 2.0.0 release we will need to make final decisions on both Java 7 and Scala 2.10. It's not really difficult to remove support for

[GitHub] spark pull request: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-29 Thread lresende
Github user lresende closed the pull request at: https://github.com/apache/spark/pull/11542 --- 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-13703][BUILD] Remove obsolete scala-2.1...

2016-03-29 Thread markhamstra
Github user markhamstra commented on the pull request: https://github.com/apache/spark/pull/11542#issuecomment-203224103 There was no consensus reached in that discussion thread. Reynold opened the thread to get a sense of how the Spark community feels about dropping Java 7 for Java

[GitHub] spark pull request: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-29 Thread lresende
Github user lresende commented on the pull request: https://github.com/apache/spark/pull/11542#issuecomment-203206216 @markhamstra The discussion seems to be getting consensus, so it's not that I am jumping the gun and creating something completely new, but reopening something that

[GitHub] spark pull request: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-29 Thread markhamstra
Github user markhamstra commented on the pull request: https://github.com/apache/spark/pull/11542#issuecomment-203184032 Re-opening is not appropriate at this time. There has been discussion on the dev list about dropping Java 7. That is a separate issue from whether Scala 2.10

[GitHub] spark pull request: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-29 Thread lresende
Github user lresende commented on the pull request: https://github.com/apache/spark/pull/11542#issuecomment-203183327 @markhamstra I had done this is the past, and with the current discussion of droping jdk 7 and scala 2.10 i think this might be useful now. Any concerns reopening

[GitHub] spark pull request: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-29 Thread markhamstra
Github user markhamstra commented on the pull request: https://github.com/apache/spark/pull/11542#issuecomment-203181027 Why has this been re-opened? --- 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: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11542#issuecomment-203180605 **[Test build #54482 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54482/consoleFull)** for PR 11542 at commit

[GitHub] spark pull request: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-29 Thread lresende
Github user lresende commented on the pull request: https://github.com/apache/spark/pull/11542#issuecomment-203179609 Jenkins 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

[GitHub] spark pull request: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11542#issuecomment-203178767 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 project does

[GitHub] spark pull request: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11542#issuecomment-203178769 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11542#issuecomment-203178766 **[Test build #54481 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54481/consoleFull)** for PR 11542 at commit

[GitHub] spark pull request: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11542#issuecomment-203178073 **[Test build #54481 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54481/consoleFull)** for PR 11542 at commit

[GitHub] spark pull request: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-29 Thread lresende
GitHub user lresende reopened a pull request: https://github.com/apache/spark/pull/11542 [SPARK-13703][BUILD] Remove obsolete scala-2.10 source files ## What changes were proposed in this pull request? Remove obsolete scala-2.10 source files ## How was this patch

[GitHub] spark pull request: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-06 Thread lresende
Github user lresende closed the pull request at: https://github.com/apache/spark/pull/11542 --- 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-13703][BUILD] Remove obsolete scala-2.1...

2016-03-06 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/11542#issuecomment-192969388 Yeah, no support has been dropped. You can close this PR. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] spark pull request: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-06 Thread markhamstra
Github user markhamstra commented on the pull request: https://github.com/apache/spark/pull/11542#issuecomment-192963213 @lresende It is not the case. In Spark 2.0, the default build will now use Scala 2.11, but we will still be supporting Scala 2.10. --- If your project is set up

[GitHub] spark pull request: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-06 Thread lresende
Github user lresende commented on the pull request: https://github.com/apache/spark/pull/11542#issuecomment-192938438 @srowen I was under the impression we were not supporting 2.10 anymore, if this is not the case I will close this PR. --- If your project is set up for it, you can

[GitHub] spark pull request: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-06 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/11542#issuecomment-192864762 Aren't these still required for Scala 2.10 support? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark pull request: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11542#issuecomment-192769369 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

[GitHub] spark pull request: [SPARK-13703][BUILD] Remove obsolete scala-2.1...

2016-03-05 Thread lresende
GitHub user lresende opened a pull request: https://github.com/apache/spark/pull/11542 [SPARK-13703][BUILD] Remove obsolete scala-2.10 source files ## What changes were proposed in this pull request? Remove obsolete scala-2.10 source files ## How was this patch