You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2020/12/15 22:21:17 UTC
[GitHub] [beam] tvalentyn opened a new pull request #13555: Revert "Do not add unnecessary experiment use_multiple_sdk_containers."
tvalentyn opened a new pull request #13555:
URL: https://github.com/apache/beam/pull/13555
Reverts apache/beam#13475
Reason: for streaming jobs new codepath using UW (https://github.com/apache/beam/blob/9f97585160fae644d9b4a7f0dca6558c82c29723/sdks/python/apache_beam/runners/dataflow/internal/apiclient.py#L276) is not yet triggered by default.
So, at this time Dataflow first need to change the behavior to not require `use_multiple_sdk_containers` for streaming pipelines, or we need to use UW codepath for all streaming pipelines in Beam.
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [beam] tvalentyn merged pull request #13555: Revert "Do not add unnecessary experiment use_multiple_sdk_containers."
Posted by GitBox <gi...@apache.org>.
tvalentyn merged pull request #13555:
URL: https://github.com/apache/beam/pull/13555
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [beam] chamikaramj commented on pull request #13555: Revert "Do not add unnecessary experiment use_multiple_sdk_containers."
Posted by GitBox <gi...@apache.org>.
chamikaramj commented on pull request #13555:
URL: https://github.com/apache/beam/pull/13555#issuecomment-745695228
LGTM
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [beam] tvalentyn commented on pull request #13555: Revert "Do not add unnecessary experiment use_multiple_sdk_containers."
Posted by GitBox <gi...@apache.org>.
tvalentyn commented on pull request #13555:
URL: https://github.com/apache/beam/pull/13555#issuecomment-745604905
R: @chamikaramj
----------------------------------------------------------------
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:
users@infra.apache.org