-
Notifications
You must be signed in to change notification settings - Fork 28.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[SPARK-13189] Cleanup build references to Scala 2.10 #11092
Changes from all commits
9fdce96
202c10b
0b5aabb
db3fdb1
282390c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -25,7 +25,7 @@ RUN apt-get update && \ | |
apt-get install -y less openjdk-7-jre-headless net-tools vim-tiny sudo openssh-server && \ | ||
rm -rf /var/lib/apt/lists/* | ||
|
||
ENV SCALA_VERSION 2.10.5 | ||
ENV SCALA_VERSION 2.11.7 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm pretty sure that this Docker image is never built / run. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Should we get rid of this then ? |
||
ENV CDH_VERSION cdh4 | ||
ENV SCALA_HOME /opt/scala-$SCALA_VERSION | ||
ENV SPARK_HOME /opt/spark | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -164,7 +164,7 @@ | |
<commons.math3.version>3.4.1</commons.math3.version> | ||
<!-- managed up from 3.2.1 for SPARK-11652 --> | ||
<commons.collections.version>3.2.2</commons.collections.version> | ||
<scala.version>2.10.5</scala.version> | ||
<scala.version>2.11.7</scala.version> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Not necessary but it doesn't hurt to avoid confusion. |
||
<scala.binary.version>2.11</scala.binary.version> | ||
<jline.version>${scala.version}</jline.version> | ||
<jline.groupid>org.scala-lang</jline.groupid> | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change is fine, but actually, it's not important to reference version numbers here. They exist because the original output was something I automatically generated from a Maven plugin. These have no functional effect and actually is not at all version specific.