iemejia commented on a change in pull request #14801:
URL: https://github.com/apache/beam/pull/14801#discussion_r634026971
##########
File path: runners/flink/flink_runner.gradle
##########
@@ -134,6 +134,7 @@ dependencies {
compile project(":runners:java-fn-execution")
compile project(":runners:java-job-service")
compile project(":sdks:java:extensions:google-cloud-platform-core")
+ compile project(":sdks:java:io:kafka")
Review comment:
I really want this to move on but I have to say that I agree with @ihji,
another reason is that adding the KafkaIO dependency to almost all runners will
bring a lot of the extra baggage that KafkaIO has including extra confluent and
spring dependencies that will leak into the user system's classpath and can
produce conflicts.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
