chamikaramj commented on code in PR #27273:
URL: https://github.com/apache/beam/pull/27273#discussion_r1244339380


##########
sdks/java/transform-service/docker-compose/docker-compose.yml:
##########
@@ -18,6 +18,9 @@ services:
     command: -port 5001
     ports:
       - "${TRANSFORM_SERVICE_PORT}:5001"
+    depends_on:
+      - expansion-service-1
+      - expansion-service-2

Review Comment:
   > Most use-cases won't need both java and python expansion services
   
   I'm hoping to use this as a generalized service that support all transforms. 
The main cost will be container download and storage. Once downloaded the 
expansion service containers can be started very quickly. Also, hoping to push 
changes to reduce size of the Java container in the future by removing the 
duplications due to using multiple shaded jars.



-- 
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]

Reply via email to