karthick-rn commented on issue #316: Add support for ZooKeeper 3.5 URL: https://github.com/apache/fluo-muchos/pull/316#issuecomment-581659678 > It seems both 3.4 and 3.5 have a lib dir, but we are not currently adding ZK/lib in accumulo-env.sh. So I suspect the ZK 3.4 dependencies are currently supplied elsewhere on the classpath. The current classpath in `accumulo-env.sh` is correct, it is not required to add ZK/lib because in 3.4 the tarball extract places `zookeeper-3.4.14.jar` in `zookeeper_home` directory and not in lib dir. This is different in 3.5 where `zookeeper-3.5.6.jar` is placed inside the lib dir. I have tested both the options & it works. a) Add ZK/lib in `accumulo-env.sh` b) Copy zookeeper*.jar from lib to `zookeeper_home` directory > If not we always have the option of making ZK 3.5 the minimum supported version of ZK for muchos (we can add a sanity check during setup). If this makes maintaining muchos simpler, it may be a good way to go. Agree > This may not be an issue for Accumulo. I think the order in which items are listed on the classpath is important for deciding which classes are used (see specification order section at the end of this doc). It makes sense & in which case I think we don't need to worry about the zookeeper*.jar that ships with Hadoop. Btw, thanks for the link.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
