[FLINK-23009] Bump up Guava in Kinesis Connector

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.14.0, 1.12.5, 1.13.2
• Component/s: Connectors / Kinesis
• Labels: pull-request-available
• Assignee: Emre Kartoglu
• Reporter: Emre Kartoglu
• Created: 16/Jun/21 08:47
• Updated: 23/Sep/21 17:52
• Resolved: 28/Jun/21 13:59

Description

Background

We maintain a copy of the Flink connector in our AWS GitHub group: https://github.com/awslabs/amazon-kinesis-connector-flink

We've recently upgraded the Guava library in our AWS copy as the version we were using was quite old and had incompatible interface differences with the later and more commonly used Guava versions. As part of this ticket we'll be applying the same changes in the Flink repo https://github.com/apache/flink/tree/master/flink-connectors/flink-connector-kinesis 

Scope

• Upgrade Guava library in pom.xml
• Switch to 3-arg version of Guava Futures.addCallback method call, as the old 2-arg version is no longer supported

Result

All existing and new tests should pass

 

 

 

Issue Links

• causes: FLINK-23151 KinesisTableApiITCase.testTableApiSourceAndSink fails on azure
• relates to: FLINK-23167 Port Kinesis Table API e2e tests to release-1.12 branch
• links to: GitHub Pull Request #16187
• links to: GitHub Pull Request #16195
• links to: GitHub Pull Request #16196
• links to: GitHub Pull Request #16292
• links to: GitHub Pull Request #16297
• links to: GitHub Pull Request #16303