You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by RussellSpitzer <gi...@git.apache.org> on 2016/08/06 05:47:46 UTC

[GitHub] spark pull request #11796: [SPARK-13579][build][test-maven] Stop building th...

Github user RussellSpitzer commented on a diff in the pull request:

    https://github.com/apache/spark/pull/11796#discussion_r73781398
  
    --- Diff: assembly/pom.xml ---
    @@ -69,6 +68,17 @@
           <artifactId>spark-repl_${scala.binary.version}</artifactId>
           <version>${project.version}</version>
         </dependency>
    +
    +    <!--
    +      Because we don't shade dependencies anymore, we need to restore Guava to compile scope so
    +      that the libraries Spark depend on have it available. We'll package the version that Spark
    +      uses (14.0.1) which is not the same as Hadoop dependencies, but works.
    +    -->
    +    <dependency>
    --- End diff --
    
    This is a problem for the Spark Cassandra Connector. The Cassandra Java Driver requires a 16.0 or greater version of guava. This necessarily means we need to shade now. This was on our roadmap anyway just wanted you to be aware.


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org