Improve CI with cancel & concurrency

apache · Mar 18, 2022 · 733df9c · 733df9c
1 parent 494c3a8
commit 733df9c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/.github/workflows/dep.yml b/.github/workflows/dep.yml
@@ -49,7 +49,7 @@ jobs:
           check-latest: false
       - name: Maven install
         run: >-
-          build/mvn -T 4 clean install -Pflink-provided,spark-provided
+          build/mvn clean install -Pflink-provided,spark-provided
           -Dmaven.javadoc.skip=true -Drat.skip=true -Dscalastyle.skip=true -Dspotless.check.skip -DskipTests
           -pl kyuubi-ctl,kyuubi-server,kyuubi-assembly -am
       - name: Check dependency list

diff --git a/.github/workflows/style.yml b/.github/workflows/style.yml
@@ -29,9 +29,6 @@ concurrency:
   group: linter-${{ github.ref }}
   cancel-in-progress: true
 
-env:
-  MAVEN_OPTS: -Dorg.slf4j.simpleLogger.defaultLogLevel=error
-
 # no install runs before style check because it is expensive, we can skip build if style fails
 # Well, sometimes when we introduce a new module, it may 'fail' the style check for missing
 # dependency we just create for other module to inherit.
@@ -55,9 +52,9 @@ jobs:
           cache: 'maven'
           check-latest: false
       - name: Scalastyle with maven
-        run: build/mvn -T 4 scalastyle:check ${{ matrix.profiles }} -pl '!:kyuubi-codecov_2.12,!:kyuubi-assembly_2.12'
+        run: build/mvn scalastyle:check ${{ matrix.profiles }} -pl '!:kyuubi-codecov_2.12,!:kyuubi-assembly_2.12'
       - name: Upload scalastyle report
         if: failure()
         run: for log in `find * -name "scalastyle-output.xml"`;  do echo "=========$log========="; grep "error" $log; done
       - name: JavaStyle with maven
-        run: build/mvn -T 4 spotless:check ${{ matrix.profiles }} -pl '!:kyuubi-codecov_2.12,!:kyuubi-assembly_2.12'
+        run: build/mvn spotless:check ${{ matrix.profiles }} -pl '!:kyuubi-codecov_2.12,!:kyuubi-assembly_2.12'