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/08 23:25:18 UTC

[GitHub] [beam] lostluck commented on pull request #15698: [BEAM-13013] Extend cross language expansion.

lostluck commented on pull request #15698:
URL: https://github.com/apache/beam/pull/15698#issuecomment-939176631


   R: @youngoli @htyleo 
   
   Separating out the External replacment to test it out a bit more first. However, this feels good to go. 
   
   Open Question: I think I might change the handful Input and Output methods on HandlerParams to instead 2 different ones, where they return a "PCollection" object of some kind consolodating all this information, since those methods are a bit messy. What do you two think?


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