jihoonson commented on a change in pull request #12095:
URL: https://github.com/apache/druid/pull/12095#discussion_r778578628
##########
File path: integration-tests/script/copy_resources_template.sh
##########
@@ -27,9 +27,16 @@ cp -r client_tls docker/client_tls
rm -rf $SHARED_DIR/docker
mkdir -p $SHARED_DIR
cp -R docker $SHARED_DIR/docker
-mvn -B dependency:copy-dependencies -DoutputDirectory=$SHARED_DIR/docker/lib
-# Make directories if they dont exist
+pushd ../
+rm -rf apache-druid-$DRUID_VERSION
+mvn -DskipTests -T1C -Danimal.sniffer.skip=true -Dcheckstyle.skip=true
-Ddruid.console.skip=true -Denforcer.skip=true -Dforbiddenapis.skip=true
-Dmaven.javadoc.skip=true -Dpmd.skip=true -Dspotbugs.skip=true install -Pdist
+tar xzf distribution/target/apache-druid-$DRUID_VERSION-bin.tar.gz
Review comment:
That's a good idea, though the extra time that this PR added in each
integration test was about 2-5 min in travis which seems not that bad. It
sounds better since it would be clearer. I added a new maven profile,
`integration-test`, that creates an exploded directory under
`distribution/target/`. Integration tests now use this new profile. In a simple
test in my machine, the `integration-test` profile took 54 sec while the `dist`
profile took 90 sec.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
