drcrallen commented on a change in pull request #6896: Add docker container for
druid
URL: https://github.com/apache/incubator-druid/pull/6896#discussion_r250317163
##########
File path: distribution/docker/Dockerfile
##########
@@ -0,0 +1,33 @@
+FROM maven:3-jdk-8 as builder
+
+COPY . /src
+WORKDIR /src
+RUN mvn install -ff -DskipTests -Dforbiddenapis.skip=true -Pdist
-Pbundle-contrib-exts
+RUN \
+ VER=$(mvn -q -Dexec.executable="echo" -Dexec.args='${project.version}'
--non-recursive exec:exec) \
+ && tar -zxf ./distribution/target/apache-druid-${VER}-bin.tar.gz -C /opt \
+ && ln -s /opt/apache-druid-${VER} /opt/druid
+
+RUN cd /opt/druid/extensions/mysql-metadata-storage \
Review comment:
Can there be no `cd` command? it would be nice if the target directories
could be specified in the commands themselves. Like `wget -O
/some/dir/something.jar`
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
