sijie commented on a change in pull request #342: Issue 338: add first draft
Docker image including community suggestions
URL: https://github.com/apache/bookkeeper/pull/342#discussion_r130796673
##########
File path: docker/Dockerfile
##########
@@ -0,0 +1,34 @@
+FROM centos:7
+MAINTAINER Apache BookKeeper <dev@bookkeeper.apache.org>
+
+ARG BK_VERSION=4.4.0
+ARG DISTRO_NAME=bookkeeper-server-${BK_VERSION}-bin
+ARG ZK_VERSION=3.5.2-alpha
Review comment:
It seems the purpose of having ZK is for `metaformat`. except that, I don't
see any reason why we need zookeeper in bookkeeper docker image. especially
bookkeeper-server package already includes zookeeper dependency -
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
