TheNeuralBit commented on a change in pull request #13211:
URL: https://github.com/apache/beam/pull/13211#discussion_r524528475
##########
File path: sdks/java/container/build.gradle
##########
@@ -86,51 +76,21 @@ licenseReport {
renderers = [new JsonReportRenderer()]
}
-def imageJavaVersion = project.hasProperty('imageJavaVersion') ?
project.findProperty('imageJavaVersion') : '8'
-docker {
Review comment:
Hm ok, I'm not sure if we have any precedent to follow for this. I don't
think it's worth going to a lot of trouble for. Instead you could just send an
FYI email to dev@ once this is merged letting people know they may need to
change :sdks:java:container:docker -> :sdks:java:container:java8:docker.
----------------------------------------------------------------
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:
us...@infra.apache.org
