Skip to content

Commit 7cfbbfa

Browse files
srowenpdeyhim
authored andcommitted
SPARK-1325. The maven build error for Spark Tools
This is just a slight variation on apache#234 and alternative suggestion for SPARK-1325. `scala-actors` is not necessary. `SparkBuild.scala` should be updated to reflect the direct dependency on `scala-reflect` and `scala-compiler`. And the `repl` build, which has the same dependencies, should also be consistent between Maven / SBT. Author: Sean Owen <[email protected]> Author: witgo <[email protected]> Closes apache#240 from srowen/SPARK-1325 and squashes the following commits: 25bd7db [Sean Owen] Add necessary dependencies scala-reflect and scala-compiler to tools. Update repl dependencies, which are similar, to be consistent between Maven / SBT in this regard too.
1 parent f3cfbc2 commit 7cfbbfa

File tree

4 files changed

+21
-1
lines changed

4 files changed

+21
-1
lines changed

pom.xml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -419,6 +419,11 @@
419419
<artifactId>scala-compiler</artifactId>
420420
<version>${scala.version}</version>
421421
</dependency>
422+
<dependency>
423+
<groupId>org.scala-lang</groupId>
424+
<artifactId>scala-reflect</artifactId>
425+
<version>${scala.version}</version>
426+
</dependency>
422427
<dependency>
423428
<groupId>org.scala-lang</groupId>
424429
<artifactId>jline</artifactId>

project/SparkBuild.scala

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -359,7 +359,9 @@ object SparkBuild extends Build {
359359
) ++ assemblySettings ++ extraAssemblySettings
360360

361361
def toolsSettings = sharedSettings ++ Seq(
362-
name := "spark-tools"
362+
name := "spark-tools",
363+
libraryDependencies <+= scalaVersion(v => "org.scala-lang" % "scala-compiler" % v ),
364+
libraryDependencies <+= scalaVersion(v => "org.scala-lang" % "scala-reflect" % v )
363365
) ++ assemblySettings ++ extraAssemblySettings
364366

365367
def graphxSettings = sharedSettings ++ Seq(

repl/pom.xml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -77,6 +77,11 @@
7777
<artifactId>scala-compiler</artifactId>
7878
<version>${scala.version}</version>
7979
</dependency>
80+
<dependency>
81+
<groupId>org.scala-lang</groupId>
82+
<artifactId>scala-reflect</artifactId>
83+
<version>${scala.version}</version>
84+
</dependency>
8085
<dependency>
8186
<groupId>org.scala-lang</groupId>
8287
<artifactId>jline</artifactId>

tools/pom.xml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,14 @@
5555
<artifactId>spark-streaming_${scala.binary.version}</artifactId>
5656
<version>${project.version}</version>
5757
</dependency>
58+
<dependency>
59+
<groupId>org.scala-lang</groupId>
60+
<artifactId>scala-reflect</artifactId>
61+
</dependency>
62+
<dependency>
63+
<groupId>org.scala-lang</groupId>
64+
<artifactId>scala-compiler</artifactId>
65+
</dependency>
5866
<dependency>
5967
<groupId>org.scalatest</groupId>
6068
<artifactId>scalatest_${scala.binary.version}</artifactId>

0 commit comments

Comments
 (0)