ihji commented on a change in pull request #14801:
URL: https://github.com/apache/beam/pull/14801#discussion_r633941572
##########
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:
Yeah, it makes sense. But I still prefer to not introduce the additional
io dependency to runners given that this change is temporary and will be
reverted soon once we fix the performance issue (I'm not totally against the
current implementation though so it's good to go if other folks don't have any
preference).
--
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]
