[GitHub] [beam] tvalentyn commented on pull request #12727: [BEAM-10844] Add experiment option prebuild_sdk_container to prebuild python sdk container with dependencies.

2020-09-23 Thread GitBox
tvalentyn commented on pull request #12727: URL: https://github.com/apache/beam/pull/12727#issuecomment-698036314 I don't have additional comments at this time so I am fine to merge and iterate. This is an automated message

[GitHub] [beam] tvalentyn commented on pull request #12727: [BEAM-10844] Add experiment option prebuild_sdk_container to prebuild python sdk container with dependencies.

2020-09-23 Thread GitBox
tvalentyn commented on pull request #12727: URL: https://github.com/apache/beam/pull/12727#issuecomment-698036221 I looked at codecov errors on the PR, I think the path that codecov reports about are covered by the integration test.