rickchengx commented on a change in pull request #4189:
URL: https://github.com/apache/zeppelin/pull/4189#discussion_r679662545



##########
File path: scripts/docker/zeppelin/build-from-distribution/Dockerfile
##########
@@ -0,0 +1,86 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more

Review comment:
       Actually, the dockerfile is different from the 
`scripts/docker/zeppelin/bin/Dockerfile` because it needs to build the docker 
image from the distribution tar.gz. But the properties file about log4j can be 
reused (copy from `scripts/docker/zeppelin/bin/` before the image building). 




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

To unsubscribe, e-mail: dev-unsubscr...@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to