This is an automated email from the ASF dual-hosted git repository.
martijnvisser pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/flink-connector-jdbc.git
The following commit(s) were added to refs/heads/main by this push:
new 90179da [FLINK-31699] Fix JDBC nightly CI failure (#37)
90179da is described below
commit 90179da68dacd6e65a399ae8e8b7c453edbfbb80
Author: MartijnVisser <[email protected]>
AuthorDate: Tue Apr 4 19:44:00 2023 +0200
[FLINK-31699] Fix JDBC nightly CI failure (#37)
* [FLINK-31699] Set correct groupId for `flink-connector-parent`
* [FLINK-31699] Update Flink to 1.17 and match other dependencies to the
version that are used in that version
* [FLINK-31699] Remove archunit to avoid issues with the weekly builds,
given that the version of Archunit can differ between Flink versions. Instead,
it will rely on flink-architecture-tests-test dependencies.
* [FLINK-31699] Run CI for PRs against Flink 1.17.0 and weekly against
Flink 1.16-SNAPSHOT, 1.17-SNAPSHOT and 1.18-SNAPSHOT
* [hotfix] Set japicmp to latest released version for this connector, which
is 3.0.0-1.16
* [hotfix] Resolve dependency convergence on `byte-buddy`
* [hotfix] To allow all Archunit rules to be evaluated without checking any
classes
---
.github/workflows/push_pr.yml | 2 +-
.github/workflows/release.yml | 31 ----------------
.github/workflows/weekly.yml | 2 +-
flink-connector-jdbc/pom.xml | 7 ----
.../src/test/resources/archunit.properties | 2 ++
pom.xml | 41 +++++++++++-----------
6 files changed, 24 insertions(+), 61 deletions(-)
diff --git a/.github/workflows/push_pr.yml b/.github/workflows/push_pr.yml
index e3a7f8b..335f108 100644
--- a/.github/workflows/push_pr.yml
+++ b/.github/workflows/push_pr.yml
@@ -25,4 +25,4 @@ jobs:
compile_and_test:
uses: apache/flink-connector-shared-utils/.github/workflows/ci.yml@ci_utils
with:
- flink_version: 1.16.1
+ flink_version: 1.17.0
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
deleted file mode 100644
index e080288..0000000
--- a/.github/workflows/release.yml
+++ /dev/null
@@ -1,31 +0,0 @@
-################################################################################
-# Licensed to the Apache Software Foundation (ASF) under one
-# or more contributor license agreements. See the NOTICE file
-# distributed with this work for additional information
-# regarding copyright ownership. The ASF licenses this file
-# to you under the Apache License, Version 2.0 (the
-# "License"); you may not use this file except in compliance
-# with the License. You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-################################################################################
-
-name: Release Build
-on:
- workflow_dispatch:
-jobs:
- compile_and_test:
- if: github.repository_owner == 'apache'
- strategy:
- matrix:
- flink: [1.16.1, 1.17.0]
- uses: apache/flink-connector-shared-utils/.github/workflows/ci.yml@ci_utils
- with:
- flink_version: ${{ matrix.flink }}
- run_dependency_convergence: false
diff --git a/.github/workflows/weekly.yml b/.github/workflows/weekly.yml
index 76c0f74..16a04e1 100644
--- a/.github/workflows/weekly.yml
+++ b/.github/workflows/weekly.yml
@@ -25,7 +25,7 @@ jobs:
if: github.repository_owner == 'apache'
strategy:
matrix:
- flink: [1.16-SNAPSHOT, 1.17-SNAPSHOT]
+ flink: [1.16-SNAPSHOT, 1.17-SNAPSHOT, 1.18-SNAPSHOT]
uses: apache/flink-connector-shared-utils/.github/workflows/ci.yml@ci_utils
with:
flink_version: ${{ matrix.flink }}
diff --git a/flink-connector-jdbc/pom.xml b/flink-connector-jdbc/pom.xml
index 5fcdb41..43ad74f 100644
--- a/flink-connector-jdbc/pom.xml
+++ b/flink-connector-jdbc/pom.xml
@@ -40,7 +40,6 @@ under the License.
<assertj.version>3.23.1</assertj.version>
<postgres.version>42.5.1</postgres.version>
<oracle.version>21.8.0.0</oracle.version>
- <byte-buddy.version>1.12.10</byte-buddy.version>
</properties>
<dependencies>
@@ -206,12 +205,6 @@ under the License.
<dependencyManagement>
<dependencies>
- <!-- For dependency convergence -->
- <dependency>
- <groupId>net.bytebuddy</groupId>
- <artifactId>byte-buddy</artifactId>
- <version>${byte-buddy.version}</version>
- </dependency>
<!-- For dependency convergence -->
<dependency>
<groupId>org.scala-lang</groupId>
diff --git a/flink-connector-jdbc/src/test/resources/archunit.properties
b/flink-connector-jdbc/src/test/resources/archunit.properties
index 15be88c..48011f9 100644
--- a/flink-connector-jdbc/src/test/resources/archunit.properties
+++ b/flink-connector-jdbc/src/test/resources/archunit.properties
@@ -29,3 +29,5 @@ freeze.store.default.allowStoreUpdate=true
#freeze.refreeze=true
freeze.store.default.path=archunit-violations
+
+archRule.failOnEmptyShould = false
diff --git a/pom.xml b/pom.xml
index 2942140..4c10d6b 100644
--- a/pom.xml
+++ b/pom.xml
@@ -20,9 +20,9 @@ under the License.
xmlns="http://maven.apache.org/POM/4.0.0";
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance";>
<parent>
- <groupId>io.github.zentol.flink</groupId>
+ <groupId>org.apache.flink</groupId>
<artifactId>flink-connector-parent</artifactId>
- <version>1.0</version>
+ <version>1.0.0</version>
</parent>
<modelVersion>4.0.0</modelVersion>
@@ -47,18 +47,17 @@ under the License.
</modules>
<properties>
- <flink.version>1.16.1</flink.version>
- <flink.shaded.version>15.0</flink.shaded.version>
+ <flink.version>1.17.0</flink.version>
+ <flink.shaded.version>16.1</flink.shaded.version>
<jackson-bom.version>2.13.4.20221013</jackson-bom.version>
<junit4.version>4.13.2</junit4.version>
- <junit5.version>5.8.1</junit5.version>
- <assertj.version>3.21.0</assertj.version>
- <archunit.version>0.22.0</archunit.version>
+ <junit5.version>5.9.1</junit5.version>
+ <assertj.version>3.23.1</assertj.version>
<testcontainers.version>1.17.2</testcontainers.version>
<mockito.version>2.21.0</mockito.version>
- <japicmp.referenceVersion>1.16.0</japicmp.referenceVersion>
+ <japicmp.referenceVersion>3.0.0-1.16</japicmp.referenceVersion>
<slf4j.version>1.7.36</slf4j.version>
<log4j.version>2.17.2</log4j.version>
@@ -300,26 +299,26 @@ under the License.
<version>2.1</version>
</dependency>
+ <!-- For dependency convergence on mockito/powermock mismatch -->
<dependency>
- <groupId>org.testcontainers</groupId>
- <artifactId>testcontainers-bom</artifactId>
- <version>${testcontainers.version}</version>
- <type>pom</type>
- <scope>import</scope>
+ <groupId>net.bytebuddy</groupId>
+ <artifactId>byte-buddy</artifactId>
+ <version>1.12.10</version>
</dependency>
+ <!-- For dependency convergence on mockito/powermock mismatch -->
<dependency>
- <groupId>com.tngtech.archunit</groupId>
- <artifactId>archunit</artifactId>
- <version>${archunit.version}</version>
- <scope>test</scope>
+ <groupId>net.bytebuddy</groupId>
+ <artifactId>byte-buddy-agent</artifactId>
+ <version>1.12.10</version>
</dependency>
<dependency>
- <groupId>com.tngtech.archunit</groupId>
- <artifactId>archunit-junit5</artifactId>
- <version>${archunit.version}</version>
- <scope>test</scope>
+ <groupId>org.testcontainers</groupId>
+ <artifactId>testcontainers-bom</artifactId>
+ <version>${testcontainers.version}</version>
+ <type>pom</type>
+ <scope>import</scope>
</dependency>
</dependencies>
