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 2022/07/15 17:47:30 UTC

[GitHub] [beam] pabloem commented on pull request #21867: Beam 12994 - Python SDK BigQuery - Promote `schemaUpdateOptions` to named arguments

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: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org