Abacn commented on code in PR #39297:
URL: https://github.com/apache/beam/pull/39297#discussion_r3575449170
##########
build.gradle.kts:
##########
@@ -1056,3 +1056,22 @@ if (project.hasProperty("testJavaVersion")) {
}
}
}
+
+subprojects {
Review Comment:
ack, is it safe to say neither 'org.lz4:lz4-java' or 'at.yawk.lz4' is Beam's
dependency (direct or transitive) based on
https://github.com/apache/beam/commit/b191394c02a14c782c9bc246d6d362ce63d5756e
? If so I'm fine with the current approach, as it would be a no-op for this
dependency set
> (Beam core + GCP dependencies + Dataflow runner)
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
