Skip to content

Commit

Permalink
HTTP103 support Flink 1.20
Browse files Browse the repository at this point in the history
Signed-off-by: davidradl <david_radley@uk.ibm.com>
  • Loading branch information
davidradl committed Jan 10, 2025
1 parent e5ddf06 commit 6db1908
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 10 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ jobs:
runs-on: ubuntu-latest
strategy:
matrix:
flink: [ "1.16.3", "1.17.2", "1.18.1"]
flink: ["1.18.1", "1.19.1", "1.20.0"]
steps:
- uses: actions/checkout@v3

Expand All @@ -38,7 +38,7 @@ jobs:
- name: Test JavaDoc
run: mvn $MAVEN_CLI_OPTS $JAVA_ADDITIONAL_OPTS javadoc:javadoc
if: startsWith(matrix.flink, '1.18')
if: startsWith(matrix.flink, '1.20')

- name: Add coverage to PR
id: jacoco
Expand All @@ -48,4 +48,4 @@ jobs:
token: ${{ secrets.GITHUB_TOKEN }}
min-coverage-overall: 40
min-coverage-changed-files: 60
if: startsWith(matrix.flink, '1.18') && github.event.pull_request.head.repo.fork == false
if: startsWith(matrix.flink, '1.20') && github.event.pull_request.head.repo.fork == false
3 changes: 2 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,8 @@

## [Unreleased]

- ignore Eclipse files in .gitignore
- Ignore Eclipse files in .gitignore
- Support Flink 1.20

## [0.17.0] - 2024-11-28

Expand Down
4 changes: 3 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,9 @@ In case of updating http-connector please see [Breaking changes](#breaking-chang
## Prerequisites
* Java 11
* Maven 3
* Flink 1.16+
* Flink 1.18+. Recommended Flink 1.20.*



## Runtime dependencies
This connector has few Flink's runtime dependencies, that are expected to be provided.
Expand Down
6 changes: 1 addition & 5 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ under the License.
<!-- IMPORTANT: If you update Flink, remember to update link to its docs in maven-javadoc-plugin <links>
section, omitting the patch part (so for 1.15.0 use 1.15). -->

<flink.version>1.16.3</flink.version>
<flink.version>1.18.1</flink.version>

<target.java.version>11</target.java.version>
<scala.binary.version>2.12</scala.binary.version>
Expand Down Expand Up @@ -303,10 +303,6 @@ under the License.
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-surefire-plugin</artifactId>
<version>3.0.0-M5</version>
<configuration>
<!-- argLine needed for Flink 1.16 and 1.17 or there are unit test errors-->
<argLine>@{argLine} --add-opens java.base/java.util=ALL-UNNAMED --add-opens java.base/java.lang=ALL-UNNAMED</argLine>
</configuration>
</plugin>

<plugin>
Expand Down

0 comments on commit 6db1908

Please sign in to comment.