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 2021/10/12 18:32:30 UTC

[GitHub] [beam] lukecwik edited a comment on pull request #15711: [BEAM-13038] Migrate to use a context object within PTransformRunnerFactory.

lukecwik edited a comment on pull request #15711:
URL: https://github.com/apache/beam/pull/15711#issuecomment-941273979


   R: @kileys @TheNeuralBit 
   
   This contains a bunch of clean-up since modifying a PTransformRunnerFactory parameter was annoying and I have a few that are coming up with a separate change so I wanted to get this out of the way since it is a large but trivial conversion. Also helps with tests a bunch since we clearly specify which parameters are being bound without needing to specify comments.


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