chamikaramj commented on a change in pull request #12505: URL: https://github.com/apache/beam/pull/12505#discussion_r488818825
########## File path: website/www/site/content/en/documentation/runtime/environments.md ########## @@ -116,8 +116,8 @@ By default, no licenses/notices are added to the docker images. To examine the containers that you built, run `docker images` from anywhere in the command line. If you successfully built all of the container images, the command prints a table like the following: ``` -REPOSITORY TAG IMAGE ID CREATED SIZE -apache/beam_java_sdk latest 16ca619d489e 2 weeks ago 550MB +REPOSITORY TAG IMAGE ID CREATED SIZE +apache/beam_java_8sdk latest 16ca619d489e 2 weeks ago 550MB Review comment: Let's make sure that the correct container name is set in the environment send to the remote SDKs that use Java transforms as cross-language transforms. (We don't have integration tests setup for this by we can just try existing Kafka/SQL x-lang examples to confirm) https://github.com/apache/beam/tree/master/sdks/python/apache_beam/examples/kafkataxi https://github.com/apache/beam/blob/master/sdks/python/apache_beam/examples/sql_taxi.py ---------------------------------------------------------------- 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