[BAHIR-241] Upgrade all connectors to Flink 1.11 #99
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
.travis.yml
Outdated
@@ -34,8 +34,8 @@ jdk: | |||
- openjdk8 | |||
|
|||
env: | |||
- FLINK_VERSION="1.10.0" SCALA_VERSION="2.11" | |||
- FLINK_VERSION="1.10.0" SCALA_VERSION="2.12" | |||
- FLINK_VERSION="1.11.1" SCALA_VERSION="2.11" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you change to same version of pom (1.11.2)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
flink-connector-flume/pom.xml
Outdated
<type>test-jar</type> | ||
<scope>test</scope> | ||
</dependency> | ||
<dependency> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you fix identation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
25dabb8
to
5d1fea9
Compare
LGTM |
5d1fea9
to
5993614
Compare
thanks @gyfora, |
Upgrades flink version to 1.11.2 for all connectors and fixes tests wherever necessary.
In addition, small improvements have been made to the Kudu connector to eliminate some deprecations