karthick-rn commented on issue #12: Update Accumulo version in Dockerfile & 
README.md
URL: https://github.com/apache/accumulo-docker/pull/12#issuecomment-611135081
 
 
   
   
   
   > @karthick-rn Since this change is really to make docker work for version 
2.1.0, if you want to keep this PR open for once we release 2.1.0 that is fine. 
We can make a separate change for now to simply update the version from alpha 
to 2.0.0.
   
   @milleruntime Sorry if I wasn't clear. This change is to make docker to work 
with Accumulo 2.0 & not 2.1. The reason for having the `accumulo-env.sh` as a 
post-script is to address the classpath issue in Accumulo 2.0 to work with ZK 
versions 3.5.x & above.  This classpath issue has been fixed in Accumulo 2.1 & 
we no longer need the post-script when we are running docker on Accumulo 2.1.  
Let me know if you are seeing any issues in Dockerfile running Accumulo 2.0? 
Also, if you still think you might need ZK 3.4.14 I'm happy to make the 
changes. Thanks

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

Reply via email to