tvalentyn commented on issue #11142: [BEAM-9510] Fixing version 
incompatibilities in base_image_requirements.txt
URL: https://github.com/apache/beam/pull/11142#issuecomment-599900258
 
 
   LGTM
   A few things out of scope of this PR:
   1) This PR fixes the incompatibility in dependencies, but the list of 
dependencies is in base_image_requirements.txt is still not exhaustive. So, 
container builds may not be repeatable over time, and there is a risk that we 
will introduce incompatible dependencies down the road and won't notice it.
   2) We can consider switching the base image to `python:slim-stretch` instead 
of `python:stretch` to reduce the size of the images: 
https://github.com/apache/beam/blob/master/sdks/python/container/Dockerfile#L20
   cc: @Hannah-Jiang 

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


With regards,
Apache Git Services

Reply via email to