ctubbsii commented on issue #12: Update Accumulo version in Dockerfile & README.md URL: https://github.com/apache/accumulo-docker/pull/12#issuecomment-611153001 > @karthick-rn I wonder if adding `ENV CLASSPATH /opt/zookeeper/lib/*` towards the end of the dockerfile instead of adding the accumulo-env.sh file would work. This should be picked up by Accumulo env, the only drawback is that we dont get control over the order of where `/opt/zookeeper/lib/*` is in the classpath. I like @keith-turner 's suggestion here. It's better than copying the env script in. @milleruntime I'm not following your use of "out-of-the-box" here. The way I see it, the Docker packaging is its own "box" (packaging). Accumulo 2.0.0 works fine, when properly configured, with ZK 3.5 or 3.6. So, it should be fine if the Docker packaging configures it that way. The only real difference is that you'll need to have the newer ZK running to use the Docker packaging of Accumulo, whereas if you were to package or deploy Accumulo manually, you could do so using a version as low as 3.4. But, I think it's fine for Docker to require newer.
---------------------------------------------------------------- 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
