eolivelli commented on a change in pull request #2433:
URL: https://github.com/apache/bookkeeper/pull/2433#discussion_r502382424



##########
File path: docker/README.md
##########
@@ -13,7 +13,7 @@ Bookkeeper needs [Zookeeper](https://zookeeper.apache.org/) 
in order to preserve
 Just like running a BookKeeper cluster in one 
machine(http://bookkeeper.apache.org/docs/latest/getting-started/run-locally/), 
you can run a standalone BookKeeper in one docker container, the command is:
 ```
 docker run -it \
-     --env JAVA_HOME=/usr/lib/jvm/jre-1.8.0 \
+     --env JAVA_HOME=/usr/lib/jvm/jdk-11 \

Review comment:
       I would like, but in current 4.11 we are using the presence of shell to 
detect jdk11 and it is not present in the.
   
   We should fix that problem, then we can use the jre.
   
   Btw I don't know a good way to detect jdk11+




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


Reply via email to