You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ignite.apache.org by GitBox <gi...@apache.org> on 2020/04/22 08:24:58 UTC

[GitHub] [ignite] Mmuzaf commented on a change in pull request #7710: IGNITE-12918 Fix Travis config: separate Java jobs

Mmuzaf commented on a change in pull request #7710:
URL: https://github.com/apache/ignite/pull/7710#discussion_r412777031



##########
File path: .travis.yml
##########
@@ -23,26 +23,32 @@ matrix:
         - echo $JVM_EXTRA_ARGS
         - echo $JDK_PROFILES
       script: mvn package -Pall-java,licenses,lgpl,checkstyle,examples $JDK_PROFILES -DskipTests -Dmaven.javadoc.skip=true -B -V
-      
-      jobs:
-        include:
-          - jdk: openjdk8
-            env:
-            - JDK_PROFILES='-Pall-scala,scala'
-          - jdk: openjdk11
-            # ignite-scalar module currently doesn't build under jdk11
-            env:
-              - JDK_PROFILES='-P!spark-2.4,!spark,!scala'
-            before_script:
-            - |
-              export JVM_EXTRA_ARGS='--illegal-access=permit
-              --add-exports=java.base/jdk.internal.misc=ALL-UNNAMED
-              --add-exports=java.base/sun.nio.ch=ALL-UNNAMED
-              --add-exports=java.management/com.sun.jmx.mbeanserver=ALL-UNNAMED
-              --add-exports=jdk.internal.jvmstat/sun.jvmstat.monitor=ALL-UNNAMED
-              --add-exports=java.base/sun.reflect.generics.reflectiveObjects=ALL-UNNAMED
-              -Djdk.tls.client.protocols=TLSv1.2'
-      
+      jdk: openjdk8
+      env:
+        - JDK_PROFILES='-Pall-scala,scala'
+      before_cache: rm -rfv $HOME/.m2/repository/org/apache/ignite
+      cache: {directories: [$HOME/.m2/repository]}
+
+    - language: java
+      os: linux
+      dist: trusty
+      install: skip
+      before_script:
+        - echo $JVM_EXTRA_ARGS
+        - echo $JDK_PROFILES
+        - |
+          export JVM_EXTRA_ARGS='--illegal-access=permit
+          --add-exports=java.base/jdk.internal.misc=ALL-UNNAMED
+          --add-exports=java.base/sun.nio.ch=ALL-UNNAMED
+          --add-exports=java.management/com.sun.jmx.mbeanserver=ALL-UNNAMED
+          --add-exports=jdk.internal.jvmstat/sun.jvmstat.monitor=ALL-UNNAMED
+          --add-exports=java.base/sun.reflect.generics.reflectiveObjects=ALL-UNNAMED
+          -Djdk.tls.client.protocols=TLSv1.2'
+      script: mvn package -Pall-java,licenses,lgpl,checkstyle,examples $JDK_PROFILES -DskipTests -Dmaven.javadoc.skip=true -B -V
+      jdk: openjdk11
+      # ignite-scalar module currently doesn't build under jdk11
+      env:
+        - JDK_PROFILES='-P!spark-2.4,!spark,!scala'

Review comment:
       I think `JDK_PROFILES` is not needed anymore since it can be directly set to an appropriate JDK profile.




----------------------------------------------------------------
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