jnturton commented on a change in pull request #2491: URL: https://github.com/apache/drill/pull/2491#discussion_r823514983
########## File path: Dockerfile ########## @@ -56,17 +56,26 @@ RUN VERSION=$(mvn -q -Dexec.executable=echo -Dexec.args='${project.version}' --n # Set the BASE_IMAGE build arg when you invoke docker build. FROM $BASE_IMAGE -ENV DRILL_HOME=/opt/drill DRILL_USER=drilluser +ENV DRILL_HOME=/opt/drill +ENV DRILL_USER=drilluser +ENV DRILL_USER_HOME=/var/lib/drill +ENV DRILL_LOG_DIR=$DRILL_USER_HOME/log +ENV DATA_VOL=/data -RUN mkdir $DRILL_HOME +RUN mkdir $DRILL_HOME $DATA_VOL + +COPY --from=build /opt/drill $DRILL_HOME Review comment: @vvysotskyi in the final commit I found and fixed the size blowup. The `RUN chmod` command was responsible for duplicating the entire Drill installation just to set file attributes, pretty lame CoW system that Docker has there. Anyway, now the `RUN chmod` is done in the intermediate container. -- 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...@drill.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org