You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shardingsphere.apache.org by du...@apache.org on 2022/11/13 12:39:13 UTC

[shardingsphere] branch master updated: Add : add workflow support for SQL parser IT (#22150)

This is an automated email from the ASF dual-hosted git repository.

duanzhengqiang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/shardingsphere.git


The following commit(s) were added to refs/heads/master by this push:
     new a189735a1db Add : add workflow support for SQL parser IT (#22150)
a189735a1db is described below

commit a189735a1db05783f550cb705065da9189a31f49
Author: 孙念君 Nianjun Sun <su...@apache.org>
AuthorDate: Sun Nov 13 20:39:07 2022 +0800

    Add : add workflow support for SQL parser IT (#22150)
---
 .github/workflows/nightly-sql-parser.yml | 38 ++++----------------------------
 sql-parser/dialect/pom.xml               | 17 ++++++++++++++
 2 files changed, 21 insertions(+), 34 deletions(-)

diff --git a/.github/workflows/nightly-sql-parser.yml b/.github/workflows/nightly-sql-parser.yml
index 2db8fbc7839..4732c822e41 100644
--- a/.github/workflows/nightly-sql-parser.yml
+++ b/.github/workflows/nightly-sql-parser.yml
@@ -26,9 +26,9 @@ env:
   MAVEN_OPTS: -Dhttp.keepAlive=false -Dmaven.wagon.http.pool=false -Dmaven.wagon.http.retryHandler.class=standard -Dmaven.wagon.http.retryHandler.count=3 -Dcheckstyle.skip=true -Dspotless.apply.skip=true -Dmaven.javadoc.skip=true -Djacoco.skip=true -Drat.skip=true
 
 jobs:
-  mysql-parse:
+  database-parse:
     if: github.repository == 'apache/shardingsphere'
-    name: CI - JDK ${{ matrix.java-version }} on ${{ matrix.os }}
+    name: CI - JDK ${{ matrix.java-version }} on ${{ matrix.os }} for ${{ matrix.database }}
     runs-on: ${{ matrix.os }}
     timeout-minutes: 90
     strategy:
@@ -36,6 +36,7 @@ jobs:
       matrix:
         os: [ ubuntu-latest ]
         java-version: [ 8 ]
+        database: [ mysql, postgresql ]
     steps:
       - uses: actions/checkout@v3
       - uses: actions/setup-java@v3
@@ -52,38 +53,7 @@ jobs:
       - name: Build prod with Maven
         run: ./mvnw -T1C -B -ntp clean install
       - name: Run SQL parser
-        run: ./mvnw -nsu -B install -f sql-parser/dialect/mysql/pom.xml -Dsql.parser.external.it.enabled=true
-      - uses: actions/upload-artifact@v3
-        with:
-          name: sql-report
-          path: /tmp/*.csv
-  postgresql-parse:
-    if: github.repository == 'apache/shardingsphere'
-    name: CI - JDK ${{ matrix.java-version }} on ${{ matrix.os }}
-    runs-on: ${{ matrix.os }}
-    timeout-minutes: 90
-    strategy:
-      fail-fast: false
-      matrix:
-        os: [ ubuntu-latest ]
-        java-version: [ 8 ]
-    steps:
-      - uses: actions/checkout@v3
-      - uses: actions/setup-java@v3
-        with:
-          distribution: 'temurin'
-          java-version: ${{ matrix.java-version }}
-      - uses: actions/cache@v3
-        with:
-          path: ~/.m2/repository
-          key: shardingsphere-maven-third-party-cache-${{ github.sha }}
-          restore-keys: |
-            shardingsphere-maven-third-party-cache-
-            shardingsphere-maven-third-party-
-      - name: Build prod with Maven
-        run: ./mvnw -T1C -B -ntp clean install
-      - name: Run SQL parser
-        run: ./mvnw -nsu -B install -f sql-parser/dialect/postgresql/pom.xml -Dsql.parser.external.it.enabled=true
+        run: ./mvnw -nsu -B install -f sql-parser/dialect/${{ matrix.database }}/pom.xml -Dsql.parser.external.it.enabled=true
       - uses: actions/upload-artifact@v3
         with:
           name: sql-report
diff --git a/sql-parser/dialect/pom.xml b/sql-parser/dialect/pom.xml
index 97d10b294dd..4460d0c07e6 100644
--- a/sql-parser/dialect/pom.xml
+++ b/sql-parser/dialect/pom.xml
@@ -51,4 +51,21 @@
             <scope>test</scope>
         </dependency>
     </dependencies>
+    
+    <build>
+        <plugins>
+            <plugin>
+                <artifactId>maven-failsafe-plugin</artifactId>
+                <executions>
+                    <execution>
+                        <id>integration-tests</id>
+                        <goals>
+                            <goal>integration-test</goal>
+                            <goal>verify</goal>
+                        </goals>
+                    </execution>
+                </executions>
+            </plugin>
+        </plugins>
+    </build>
 </project>