gianm commented on a change in pull request #8400: enable unit tests with JDK11
URL: https://github.com/apache/incubator-druid/pull/8400#discussion_r318699240
##########
File path: .travis.yml
##########
@@ -47,31 +47,40 @@ env:
# output. To compensate, use travis_wait to extend the timeout.
install: MAVEN_OPTS='-Xmx3000m' travis_wait 15 ${MVN} clean install -q -ff
${MAVEN_SKIP} ${MAVEN_SKIP_TESTS} -T1C
-matrix:
- include:
- - name: "java 11 build"
- jdk: openjdk11
- script: ${MVN} test -pl '!web-console' ${MAVEN_SKIP}
+stages:
Review comment:
On the contrary, I do think the longer CI time is problematic when it takes
a review that could happen in a 2–3 cycles over the course of a day, and
stretches it over multiple days. For this reason it's best to have it be as
short as possible.
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
