Skip to content

Conversation

@srowen
Copy link
Member

@srowen srowen commented Feb 18, 2017

What changes were proposed in this pull request?

Go back to selecting source/target 1.7 for Scala 2.10 builds, because the SBT-based build for 2.10 won't work otherwise.

How was this patch tested?

Existing tests, but, we need to verify this vs what the SBT build would exactly run on Jenkins

@SparkQA
Copy link

SparkQA commented Feb 18, 2017

Test build #73104 has finished for PR 16983 at commit fd84d8e.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@srowen
Copy link
Member Author

srowen commented Feb 18, 2017

This change does avoid the error at #16871 (comment) when I run the same commands that the Jenkins job does

 ./dev/change-scala-version.sh 2.10
build/sbt -Phive -Phive-thriftserver -Pyarn -Phadoop-2.3 -Pkinesis-asl -Pspark-ganglia-lgpl -Pmesos -Dscala-2.10 compile test:compile

When I undo it it fails again. When back online I'll check that the changed build runs to completion with SBT.

@ericl
Copy link
Contributor

ericl commented Feb 18, 2017

LGTM if the build finishes

@srowen
Copy link
Member Author

srowen commented Feb 19, 2017

Merged to master

@asfgit asfgit closed this in df3cbe3 Feb 19, 2017
@srowen srowen deleted the SPARK-19550.3 branch February 19, 2017 17:37
Yunni pushed a commit to Yunni/spark that referenced this pull request Feb 27, 2017
…still

## What changes were proposed in this pull request?

Go back to selecting source/target 1.7 for Scala 2.10 builds, because the SBT-based build for 2.10 won't work otherwise.

## How was this patch tested?

Existing tests, but, we need to verify this vs what the SBT build would exactly run on Jenkins

Author: Sean Owen <sowen@cloudera.com>

Closes apache#16983 from srowen/SPARK-19550.3.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants