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

lzljs3620320 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-paimon.git


The following commit(s) were added to refs/heads/master by this push:
     new 2b92827d5 [hotfix] Add flink1.17 e2e test ci check (#804)
2b92827d5 is described below

commit 2b92827d58d2f2aff6174847fd2f6f0a207d1d26
Author: Kerwin <[email protected]>
AuthorDate: Mon Apr 3 10:31:57 2023 +0800

    [hotfix] Add flink1.17 e2e test ci check (#804)
---
 .github/workflows/e2e-tests-1.16.yml                |  4 ++--
 ...{e2e-tests-1.16.yml => e2e-tests-1.17-jdk11.yml} | 21 +++++++++++----------
 .../{e2e-tests-1.16.yml => e2e-tests-1.17.yml}      |  6 +++---
 paimon-e2e-tests/pom.xml                            | 18 +++++++++++++-----
 .../org/apache/paimon/hive/Hive23CatalogITCase.java |  8 +++-----
 .../org/apache/paimon/hive/Hive31CatalogITCase.java |  8 +++-----
 pom.xml                                             |  4 ++--
 7 files changed, 37 insertions(+), 32 deletions(-)

diff --git a/.github/workflows/e2e-tests-1.16.yml 
b/.github/workflows/e2e-tests-1.16.yml
index 9a9cf4435..39af919e1 100644
--- a/.github/workflows/e2e-tests-1.16.yml
+++ b/.github/workflows/e2e-tests-1.16.yml
@@ -41,7 +41,7 @@ jobs:
           java-version: ${{ env.JDK_VERSION }}
           distribution: 'adopt'
       - name: Build Flink 1.16
-        run: ./mvnw clean install -DskipTests
+        run: ./mvnw clean install -DskipTests -Pflink-1.16
       - name: Test Flink 1.16
         timeout-minutes: 60
         run: |
@@ -49,4 +49,4 @@ jobs:
           . .github/workflows/utils.sh
           jvm_timezone=$(random_timezone)
           echo "JVM timezone is set to $jvm_timezone"
-          ./mvnw test -pl paimon-e2e-tests -Duser.timezone=$jvm_timezone
+          ./mvnw test -pl paimon-e2e-tests -Duser.timezone=$jvm_timezone 
-Pflink-1.16
diff --git a/.github/workflows/e2e-tests-1.16.yml 
b/.github/workflows/e2e-tests-1.17-jdk11.yml
similarity index 82%
copy from .github/workflows/e2e-tests-1.16.yml
copy to .github/workflows/e2e-tests-1.17-jdk11.yml
index 9a9cf4435..1590b0926 100644
--- a/.github/workflows/e2e-tests-1.16.yml
+++ b/.github/workflows/e2e-tests-1.17-jdk11.yml
@@ -16,20 +16,22 @@
 # limitations under the License.
 
################################################################################
 
-name: End to End Tests Flink 1.16
+name: End to End Tests Flink 1.17 on JDK 11
 
 on:
-  push:
-  pull_request:
-    paths-ignore:
-      - 'docs/**'
-      - '**/*.md'
+  issue_comment:
+    types: [created, edited, deleted]
+
+  # daily run
+  schedule:
+    - cron: "0 0 * * *"
 
 env:
-  JDK_VERSION: 8
+  JDK_VERSION: 11
 
 jobs:
   build:
+    if: contains(github.event.comment.html_url, '/pull/') && 
contains(github.event.comment.body, '/jdk11')
     runs-on: ubuntu-latest
 
     steps:
@@ -40,10 +42,9 @@ jobs:
         with:
           java-version: ${{ env.JDK_VERSION }}
           distribution: 'adopt'
-      - name: Build Flink 1.16
+      - name: Build Flink 1.17
         run: ./mvnw clean install -DskipTests
-      - name: Test Flink 1.16
-        timeout-minutes: 60
+      - name: Test Flink 1.17
         run: |
           # run tests with random timezone to find out timezone related bugs
           . .github/workflows/utils.sh
diff --git a/.github/workflows/e2e-tests-1.16.yml 
b/.github/workflows/e2e-tests-1.17.yml
similarity index 94%
copy from .github/workflows/e2e-tests-1.16.yml
copy to .github/workflows/e2e-tests-1.17.yml
index 9a9cf4435..bf93e7c4e 100644
--- a/.github/workflows/e2e-tests-1.16.yml
+++ b/.github/workflows/e2e-tests-1.17.yml
@@ -16,7 +16,7 @@
 # limitations under the License.
 
################################################################################
 
-name: End to End Tests Flink 1.16
+name: End to End Tests Flink 1.17
 
 on:
   push:
@@ -40,9 +40,9 @@ jobs:
         with:
           java-version: ${{ env.JDK_VERSION }}
           distribution: 'adopt'
-      - name: Build Flink 1.16
+      - name: Build Flink 1.17
         run: ./mvnw clean install -DskipTests
-      - name: Test Flink 1.16
+      - name: Test Flink 1.17
         timeout-minutes: 60
         run: |
           # run tests with random timezone to find out timezone related bugs
diff --git a/paimon-e2e-tests/pom.xml b/paimon-e2e-tests/pom.xml
index 5ae584b49..943f3da05 100644
--- a/paimon-e2e-tests/pom.xml
+++ b/paimon-e2e-tests/pom.xml
@@ -175,12 +175,10 @@ under the License.
     <profiles>
         <!-- Activate these profiles with -Pflink-x.xx to build and test 
against different Flink versions -->
         <profile>
-            <id>flink-1.14</id>
+            <id>flink-1.16</id>
             <properties>
-                <test.flink.main.version>1.14</test.flink.main.version>
-                <test.flink.version>1.14.6</test.flink.version>
-                
<flink.sql.connector.kafka>flink-sql-connector-kafka_${scala.binary.version}</flink.sql.connector.kafka>
-                
<flink.sql.connector.hive>flink-sql-connector-hive-2.3.6_${scala.binary.version}</flink.sql.connector.hive>
+                <test.flink.main.version>1.16</test.flink.main.version>
+                <test.flink.version>1.16.1</test.flink.version>
             </properties>
         </profile>
 
@@ -193,5 +191,15 @@ under the License.
                 
<flink.sql.connector.hive>flink-sql-connector-hive-2.3.6_${scala.binary.version}</flink.sql.connector.hive>
             </properties>
         </profile>
+
+        <profile>
+            <id>flink-1.14</id>
+            <properties>
+                <test.flink.main.version>1.14</test.flink.main.version>
+                <test.flink.version>1.14.6</test.flink.version>
+                
<flink.sql.connector.kafka>flink-sql-connector-kafka_${scala.binary.version}</flink.sql.connector.kafka>
+                
<flink.sql.connector.hive>flink-sql-connector-hive-2.3.6_${scala.binary.version}</flink.sql.connector.hive>
+            </properties>
+        </profile>
     </profiles>
 </project>
diff --git 
a/paimon-hive/paimon-hive-connector-2.3/src/test/java/org/apache/paimon/hive/Hive23CatalogITCase.java
 
b/paimon-hive/paimon-hive-connector-2.3/src/test/java/org/apache/paimon/hive/Hive23CatalogITCase.java
index 8c143617b..2df418e37 100644
--- 
a/paimon-hive/paimon-hive-connector-2.3/src/test/java/org/apache/paimon/hive/Hive23CatalogITCase.java
+++ 
b/paimon-hive/paimon-hive-connector-2.3/src/test/java/org/apache/paimon/hive/Hive23CatalogITCase.java
@@ -138,11 +138,9 @@ public class Hive23CatalogITCase extends 
HiveCatalogITCaseBase {
         assertThatThrownBy(() -> tEnv.executeSql("ALTER TABLE 
alter_failed_table SET ('aa'='bb')"))
                 .isInstanceOf(TableException.class)
                 .hasMessage(
-                        String.format(
-                                "Could not execute "
-                                        + "ALTER TABLE 
my_alter_hive.default.alter_failed_table "
-                                        + "SET (aa: [bb], path: 
[%sdefault.db/alter_failed_table])",
-                                path));
+                        "Could not execute "
+                                + "ALTER TABLE 
my_alter_hive.default.alter_failed_table\n"
+                                + "  SET 'aa' = 'bb'");
 
         assertTrue(
                 new SchemaManager(
diff --git 
a/paimon-hive/paimon-hive-connector-3.1/src/test/java/org/apache/paimon/hive/Hive31CatalogITCase.java
 
b/paimon-hive/paimon-hive-connector-3.1/src/test/java/org/apache/paimon/hive/Hive31CatalogITCase.java
index 7c6b3f068..26d24c74b 100644
--- 
a/paimon-hive/paimon-hive-connector-3.1/src/test/java/org/apache/paimon/hive/Hive31CatalogITCase.java
+++ 
b/paimon-hive/paimon-hive-connector-3.1/src/test/java/org/apache/paimon/hive/Hive31CatalogITCase.java
@@ -138,11 +138,9 @@ public class Hive31CatalogITCase extends 
HiveCatalogITCaseBase {
         assertThatThrownBy(() -> tEnv.executeSql("ALTER TABLE 
alter_failed_table SET ('aa'='bb')"))
                 .isInstanceOf(TableException.class)
                 .hasMessage(
-                        String.format(
-                                "Could not execute "
-                                        + "ALTER TABLE 
my_alter_hive.default.alter_failed_table "
-                                        + "SET (aa: [bb], path: 
[%sdefault.db/alter_failed_table])",
-                                path));
+                        "Could not execute "
+                                + "ALTER TABLE 
my_alter_hive.default.alter_failed_table\n"
+                                + "  SET 'aa' = 'bb'");
 
         assertTrue(
                 new SchemaManager(
diff --git a/pom.xml b/pom.xml
index b18f895ca..7f6f4ddf7 100644
--- a/pom.xml
+++ b/pom.xml
@@ -114,8 +114,8 @@ under the License.
         <!-- Can be set to any value to reproduce a specific build. -->
         <test.randomization.seed/>
         <test.unit.pattern>**/*Test.*</test.unit.pattern>
-        <test.flink.main.version>1.16</test.flink.main.version>
-        <test.flink.version>1.16.1</test.flink.version>
+        <test.flink.main.version>1.17</test.flink.main.version>
+        <test.flink.version>1.17.0</test.flink.version>
 
         <janino.version>3.0.11</janino.version>
         <mockito.version>3.4.6</mockito.version>

Reply via email to