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

    https://github.com/apache/spark/pull/480#discussion_r11857494
  
    --- Diff: pom.xml ---
    @@ -892,10 +1081,11 @@
             <dependency>
               <groupId>org.apache.zookeeper</groupId>
               <artifactId>zookeeper</artifactId>
    +          <version>3.4.5</version>
    --- End diff --
    
    I see, this declaration is just there to make zookeeper "provided" in order 
to not conflict with a cluster's copy/version of ZK from Hadoop. Hadoop version 
kind of matters in that sense. I wonder if this is actually best left 
unspecified then, since there is no intent to enforce any particular version of 
ZK, just make sure it is not in the artifact when built with this profile? 


---
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 [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to