dermasmid commented on code in PR #29944: URL: https://github.com/apache/beam/pull/29944#discussion_r1451745109
########## sdks/typescript/src/apache_beam/transforms/localParDo.ts: ########## @@ -0,0 +1,285 @@ +/* Review Comment: im not sure its a good idea to copy the file, instead i think we should extend Pardo.ts to support both. the question is if we even want both.... this entire PR is more of a proposal, i wanted to get more input on this idea before really implementing it cleanly -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
