GitHub user sadhen opened a pull request:
https://github.com/apache/spark/pull/22280
[SPARK-25235] [Build] [SHELL] [FOLLOWUP] Fix repl compile for 2.12
## What changes were proposed in this pull request?
Error messages from
https://amplab.cs.berkeley.edu/jenkins/view/Spark%20QA%20Test/job/spark-master-test-maven-hadoop-2.7-ubuntu-scala-2.12/183/
```
[INFO] --- scala-maven-plugin:3.2.2:compile (scala-compile-first) @
spark-repl_2.12 ---
[INFO] Using zinc server for incremental compilation
[warn] Pruning sources from previous analysis, due to incompatible
CompileSetup.
[info] Compiling 6 Scala sources to
/home/jenkins/workspace/spark-master-test-maven-hadoop-2.7-ubuntu-scala-2.12/repl/target/scala-2.12/classes...
[error]
/home/jenkins/workspace/spark-master-test-maven-hadoop-2.7-ubuntu-scala-2.12/repl/scala-2.11/src/main/scala/org/apache/spark/repl/SparkILoopInterpreter.scala:80:
overriding lazy value importableSymbolsWithRenames in class ImportHandler of
type List[(this.intp.global.Symbol, this.intp.global.Name)];
[error] lazy value importableSymbolsWithRenames needs `override' modifier
[error] lazy val importableSymbolsWithRenames: List[(Symbol, Name)] =
{
[error] ^
[warn]
/home/jenkins/workspace/spark-master-test-maven-hadoop-2.7-ubuntu-scala-2.12/repl/src/main/scala/org/apache/spark/repl/SparkILoop.scala:53:
variable addedClasspath in class ILoop is deprecated (since 2.11.0): use
reset, replay or require to update class path
[warn] if (addedClasspath != "") {
[warn] ^
[warn]
/home/jenkins/workspace/spark-master-test-maven-hadoop-2.7-ubuntu-scala-2.12/repl/src/main/scala/org/apache/spark/repl/SparkILoop.scala:54:
variable addedClasspath in class ILoop is deprecated (since 2.11.0): use
reset, replay or require to update class path
[warn] settings.classpath append addedClasspath
[warn] ^
[warn] two warnings found
[error] one error found
[error] Compile failed at Aug 29, 2018 5:28:22 PM [0.679s]
```
Readd the profile for `scala-2.12`. Using `-Pscala-2.12` will overrides
`extra.source.dir` and `extra.testsource.dir` with two non-exist directories.
## How was this patch tested?
```
dev/change-scala-version.sh 2.12
mvn -Pscala-2.12 -DskipTests compile install
```
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/sadhen/spark SPARK_24785_FOLLOWUP
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/22280.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #22280
----
commit de4f543a98834b519a4ac0b5f638e176e1c06967
Author: å¿å¬ <rendong@...>
Date: 2018-08-30T07:21:46Z
fix repl compile for 2.12
----
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
