Skip to content

Commit 0cfd9f8

Browse files
authored
Merge pull request #52 from getindata/rc-0.9.0_testRunFix
rc-0.9.0_testRunFix - fix publish build #2
2 parents c18d960 + 1ef92ba commit 0cfd9f8

File tree

3 files changed

+6
-3
lines changed

3 files changed

+6
-3
lines changed

.github/workflows/publish.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -27,11 +27,11 @@ jobs:
2727
cache: maven
2828

2929
- name: Build
30-
run: mvn $MAVEN_CLI_OPTS $JAVA_ADDITIONAL_OPTS -Dflink.version=1.15.0 compile
30+
run: mvn $MAVEN_CLI_OPTS $JAVA_ADDITIONAL_OPTS compile
3131

3232
- name: Tests
3333
run: |
34-
mvn $MAVEN_CLI_OPTS $JAVA_ADDITIONAL_OPTS -Dflink.version=1.15.0 test integration-test
34+
mvn $MAVEN_CLI_OPTS $JAVA_ADDITIONAL_OPTS test integration-test
3535
3636
- name: Set up Apache Maven Central
3737
uses: actions/setup-java@v3

README.md

+3
Original file line numberDiff line numberDiff line change
@@ -335,6 +335,9 @@ Project build command: `mvn package`. </br>
335335
Detailed test report can be found under `target/site/jacoco/index.xml`.
336336

337337
## Demo application
338+
This demo works only for Flink-1.15.x even though connector can be used with Flink 1.16.
339+
This is probably due to bug with SQLClient [FLINK-31018](https://issues.apache.org/jira/browse/FLINK-31018)
340+
338341
You can test this connector using simple mock http server provided with this repository and Flink SQL-client.
339342
The mock server can be started from IDE (currently only this way) by running `HttpStubApp::main` method.
340343
It will start HTTP server listening on `http://localhost:8080/client`

pom.xml

+1-1
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ under the License.
6868

6969
<!-- IMPORTANT: If you update Flink, remember to update link to its docs in maven-javadoc-plugin <links>
7070
section, omitting the patch part (so for 1.15.0 use 1.15). -->
71-
<flink.version>[1.15.0,)</flink.version>
71+
<flink.version>1.15.0</flink.version>
7272

7373
<target.java.version>11</target.java.version>
7474
<scala.binary.version>2.12</scala.binary.version>

0 commit comments

Comments
 (0)