Github user jkleckner commented on the pull request:

    https://github.com/apache/spark/pull/4780#issuecomment-77195569
  
    Well, [sbt-assembly 
rename](http://stackoverflow.com/questions/24596914/sbt-assembly-rename-class-with-merge-conflicts-shade)
 does not perform a shade operation.
    
    So I guess the options are:
    * We manually shade/rename the functions.
    * We switch to maven for packaging instead of sbt in order to do the shading
    * Fix Spark via one of:
      * Since Spark does use maven, it could shade (relocate) the use of 
HyperLogLog
      * Include the fastutil package (this patch) rejected strongly for 
understandable reasons
      * Fix the user class path first mechanism to properly find clearspring in 
our jar
    
    Did I miss any options?


---
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.
---

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

Reply via email to