pabloem commented on PR #21867: URL: https://github.com/apache/beam/pull/21867#issuecomment-1185766365
hi @waltage I am so sorry to be so delayed on this. I prefer to avoid adding new parameters to the transform - it already has lots and lots of parameters, and it's hard to be sure which ones to fill up. I can see it's valuable to be able to validate the parameter directly, so that's fair. (although we could potentially validate it within`additional_bq_parameters`) Can you tell me more about the reasoning to add it? -- 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]
