Jackie-Jiang commented on a change in pull request #4200: Adding script to build and publish docker image URL: https://github.com/apache/incubator-pinot/pull/4200#discussion_r283034168
########## File path: docker/Dockerfile ########## @@ -0,0 +1,35 @@ +FROM openjdk:8u181-jdk-stretch + +LABEL MAINTAINER=fx19880...@gmail.com + +ARG PINOT_BRANCH=master +ARG PINOT_GIT_URL="https://github.com/apache/incubator-pinot.git"; +RUN echo "Trying to build Pinot from [ ${PINOT_GIT_URL} ] on branch [ ${PINOT_BRANCH} ]" +ENV PINOT_HOME=/opt/pinot +ENV PINOT_BUILD_DIR=/opt/pinot-build + +# extra dependency for running launcher +RUN apt-get update && apt-get install -y --no-install-recommends \ + vim wget curl git maven\ + && rm -rf /var/lib/apt/lists/* Review comment: Can you re-format this part? ---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
