dimas-b commented on code in PR #1897:
URL: https://github.com/apache/polaris/pull/1897#discussion_r2153005510


##########
.github/workflows/gradle.yml:
##########
@@ -33,65 +33,99 @@ on:
     branches: [ "main" ]
 
 jobs:
+
+  # TODO remove this job in a future PR
   build:
 
     runs-on: ubuntu-latest
     permissions:
       contents: read
+    steps:
+      - name: Always succeeds
+        run: echo "Success!"
 
+  unit-tests:
+    name: Unit Tests
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
     steps:
       - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4
       - name: Set up JDK 21
         uses: actions/setup-java@c5195efecf7bdfc987ee8bae7a71cb8b11521c00 # v4
         with:
           java-version: '21'
           distribution: 'temurin'
-
-      # Configure Gradle for optimal use in GiHub Actions, including caching 
of downloaded dependencies.
-      # See: https://github.com/gradle/actions/blob/main/setup-gradle/README.md
       - name: Setup Gradle
         uses: 
gradle/actions/setup-gradle@ac638b010cf58a27ee6c972d7336334ccaf61c96 # v4
         with:
-          # The setup-gradle action fails, if the wrapper is not using the 
right version or is not present.
-          # Our `gradlew` validates the integrity of the `gradle-wrapper.jar`, 
so it's safe to disable this.
           validate-wrappers: false
-
-      - name: Code style checks and tests
-        run: ./gradlew --continue check
-
-      - name: Check Maven publication
-        run: ./gradlew publishToMavenLocal sourceTarball
-
+      - name: Run unit tests
+        run: |
+          ./gradlew check sourceTarball distTar distZip publishToMavenLocal \
+            -x :polaris-quarkus-service:test \
+            -x :polaris-quarkus-admin:test \
+            -x intTest --continue
       - name: Archive test results
         uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 
# v4
         if: always()
         with:
-          name: upload-test-artifacts
+          name: upload-unit-test-artifacts
           path: |
             **/build/test-results/**
 
-      - name: Stop Gradle daemons
-        run: ./gradlew --stop
-
-      # Ensure that the build works properly when building against the "latest 
greatest" Java version
-      - name: Set up JDK 23
+  quarkus-tests:
+    name: Quarkus Tests
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+    steps:
+      - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4
+      - name: Set up JDK 21
         uses: actions/setup-java@c5195efecf7bdfc987ee8bae7a71cb8b11521c00 # v4
         with:
-          java-version: '23'
+          java-version: '21'
           distribution: 'temurin'
-      - name: Show Java version
-        run: java -version
-      - name: Clean
-        run: ./gradlew clean
-      - name: Build
-        run: ./gradlew compileAll
-      - name: Run selected tests
-        run: ./gradlew :polaris-quarkus-service:intTest
+      - name: Setup Gradle
+        uses: 
gradle/actions/setup-gradle@ac638b010cf58a27ee6c972d7336334ccaf61c96 # v4
+        with:
+          validate-wrappers: false
+      - name: Run Quarkus tests
+        run: |
+          ./gradlew \
+            :polaris-quarkus-service:test \

Review Comment:
   I'd assume `intTest` takes the longest time in general, so it might be 
preferable to break it down if the goal is to reduce CI time :thinking: 



-- 
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: issues-unsubscr...@polaris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to