[GitHub] [accumulo-docker] keith-turner commented on a change in pull request #12: Update Accumulo version in Dockerfile & README.md

[GitBox <gi...@apache.org>]

keith-turner commented on a change in pull request #12: Update Accumulo version in Dockerfile & README.md
URL: https://github.com/apache/accumulo-docker/pull/12#discussion_r406304659
 
 

 ##########
 File path: Dockerfile
 ##########
 @@ -73,11 +73,14 @@ RUN tar xzf hadoop.tar.gz -C /tmp/
 RUN tar xzf zookeeper.tar.gz -C /tmp/
 
 RUN mv /tmp/hadoop-$HADOOP_VERSION /opt/hadoop
-RUN mv /tmp/zookeeper-$ZOOKEEPER_VERSION /opt/zookeeper
+RUN mv /tmp/apache-zookeeper-$ZOOKEEPER_VERSION-bin /opt/zookeeper
 RUN mv /tmp/accumulo-$ACCUMULO_VERSION /opt/accumulo
 
 RUN /opt/accumulo/bin/accumulo-util build-native
 
+# The below line is required for Accumulo 2.0 to work with ZK 3.5 & above. 
 
 Review comment:
   ```suggestion
   # The below line is required for Accumulo 2.0 to work with ZK 3.5 & above.  This will not be needed for Accumulo 2.1
   ```

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services