[GitHub] [incubator-druid] xvrl commented on a change in pull request #8400: enable unit tests with JDK11

[GitBox <gi...@apache.org>]

xvrl commented on a change in pull request #8400: enable unit tests with JDK11
URL: https://github.com/apache/incubator-druid/pull/8400#discussion_r318309802
 
 

 ##########
 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:
   agree the total time is not the greatest, which is why I brought it up in my PR description. However, it does seem wasteful to run integration tests if basic unit tests don't even pass.
   Let me try removing stages to see how long it takes

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org