damccorm commented on PR #23356: URL: https://github.com/apache/beam/pull/23356#issuecomment-1259696736
I haven't done a detailed review, but this looks like the right approach if we want to do this. Lets first reach consensus on if/where we want to add kwargs (or a similar concept) though so that we can avoid making multiple potentially breaking changes -- 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]
