dannycranmer commented on code in PR #188:
URL:
https://github.com/apache/flink-connector-aws/pull/188#discussion_r2065697710
##########
flink-connector-aws/flink-connector-aws-kinesis-streams/pom.xml:
##########
@@ -52,6 +60,11 @@ under the License.
<artifactId>kinesis</artifactId>
</dependency>
+ <dependency>
+ <groupId>software.amazon.kinesis</groupId>
+ <artifactId>amazon-kinesis-client</artifactId>
Review Comment:
This is a shame, can we do this without KCL? It's a heavy weight dependency
for the 1 class(?) needed for deaggregation. We have been talking about
removing this dependency with the new connector for years, but never really
solved this bit. Can we ask Kinesis to publish a separate library for this? Or,
we could just copy that file (assuming it is still a small asking of code)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
