[GitHub] [linkis] GuoPhilipse commented on a diff in pull request #4243: Git action adds spark compilation

["GuoPhilipse (via GitHub)" <gi...@apache.org>]

GuoPhilipse commented on code in PR #4243:
URL: https://github.com/apache/linkis/pull/4243#discussion_r1116432108


##########
.github/workflows/build-backend.yml:
##########
@@ -37,6 +36,31 @@ jobs:
     - name: Build backend by maven
       run:
         ./mvnw clean package
+  build-spark:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        installSpark: [ 'init', '3.2','2.4' ]
+    steps:
+    - name: Checkout
+      uses: actions/checkout@v2
+    - name: Set up JDK 8
+      uses: actions/setup-java@v2
+      with:
+       distribution: 'temurin'
+       java-version: 8
+    - if: ${{ matrix.installSpark == 'init' }}
+      name: build-spark on init
+      run:
+        ./mvnw -N install
+    - if: ${{ matrix.installSpark == '3.2' }}
+      name: build-spark on 3.2
+      run:
+        ./mvnw clean install -Pspark-3.2 -Phadoop-3.3 -Dmaven.test.skip=true

Review Comment:
   for we now have added spark3-hadoop3、 spark2-hadoop3 、spark2-hadoop2(default) ,do we also need to add spark3-hadoop2?



-- 
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: notifications-unsubscribe@linkis.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@linkis.apache.org
For additional commands, e-mail: notifications-help@linkis.apache.org