paul-rogers commented on PR #12957:
URL: https://github.com/apache/druid/pull/12957#issuecomment-1226027631
The fix is good, but incurs the cost of building the IT tools every time.
(Bad enough we have to rebuild the image.) A better fix is in `.travis.yml`,
line 49:
```text
install...' && MAVEN_OPTS='-Xmx3000m' travis_wait 15 ${MVN} clean install -q
-ff -pl '!distribution,!:druid-it-tools,!:druid-it-image,!:druid-it-cases'
${MAVEN_SKIP} ${MAVEN_SKIP_TESTS} -T1C && ${MVN} install -q -ff -pl
'distribution' ${MAVEN_SKIP} ${MAVEN_SKIP_TESTS}
```
Remove the `druid-it-tools` exclusion:
```text
install...' && MAVEN_OPTS='-Xmx3000m' travis_wait 15 ${MVN} clean install -q
-ff -pl '!distribution,!:druid-it-image,!:druid-it-cases' ${MAVEN_SKIP}
${MAVEN_SKIP_TESTS} -T1C && ${MVN} install -q -ff -pl 'distribution'
${MAVEN_SKIP} ${MAVEN_SKIP_TESTS}
```
Also, it seems that the above does build a distribution, so we can skip that
step in `it.sh`. So, `it.sh` would change to:
```text
"travis" )
...
$0 image
$0 test $1
$0 tail $1
```
That is, remove `dist` (seems to already be done) and `tools` (since that
will now be done in the Maven build.)
--
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]
