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 2022/09/07 14:15:59 UTC

[GitHub] [beam] TheNeuralBit commented on pull request #22654: Remove subprocess.PIPE usage by using a temp file

TheNeuralBit commented on PR #22654:
URL: https://github.com/apache/beam/pull/22654#issuecomment-1239451833

   > @robertwb I think you had concerns about this change.
   > 
   > Do you think this should be reverted ?
   
   I'm curious what the concerns were. My understanding was that this was a net positive, orthogonal to the grpc issue.


-- 
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: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org