Github user Leemoonsoo commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/928
  
    @rawkintrevo Thanks for the explanation. Approach 1 to lower the barrier 
and keep maintenance minimum make sense.
    
    **License**
    Following new dependencies are introduced. They need to be addressed in 
https://github.com/apache/incubator-zeppelin/tree/master/zeppelin-distribution/src/bin_license
    ```
    [INFO] |  +- it.unimi.dsi:fastutil:jar:7.0.11:compile
    [INFO] |  \- com.tdunning:t-digest:jar:3.1:compile
    [INFO] |  +- com.github.scopt:scopt_2.10:jar:3.3.0:compile
    [INFO] |  \- com.github.haifengl:smile-plot:jar:1.1.0:compile
    [INFO] |     +- com.github.haifengl:smile-core:jar:1.1.0:compile
    [INFO] |     |  +- com.github.haifengl:smile-data:jar:1.1.0:compile
    [INFO] |     |  +- com.github.haifengl:smile-math:jar:1.1.0:compile
    [INFO] |     |  \- com.github.haifengl:smile-graph:jar:1.1.0:compile
    [INFO] |     \- org.swinglabs:swingx:jar:1.6.1:compile
    [INFO] |        +- com.jhlabs:filters:jar:2.0.235:compile
    [INFO] |        \- org.swinglabs:swing-worker:jar:1.1:compile
    [INFO] |     +- com.thoughtworks.xstream:xstream:jar:1.4.4:compile
    [INFO] |     |  +- xmlpull:xmlpull:jar:1.1.3.1:compile
    [INFO] |     |  \- xpp3:xpp3_min:jar:1.1.4c:compile
    ```
    Check dependency tree with `mvn package dependency:tree` command
    
    **Dependency confliction**
    Following dependency versions are different from dependency brought by 
Spark.
    
    Mahout
    ```
    com.esotericsoftware.kryo:kryo:jar:2.24:compile
    com.fasterxml.jackson.core:jackson-core:jar:2.7.2:compile
    ```
    
    Spark (1.6)
    ```
    com.esotericsoftware.kryo:kryo:jar:2.21:compile
    com.fasterxml.jackson.core:jackson-core:jar:2.4.4:compile
    ```
    Can all conflicted dependencies be excluded ?
    
    
    **Packaging**
    If mahout requires to be loaded in spark executor's classpath, then adding 
mahout dependency in pom.xml will not be enough to work with Spark cluster. 
Could you clarify if mahout need to be loaded in spark executor?



---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to