WIP: Add DefaultProjectId, DefaultDatasetID and Parameters QueryOptions to go bigqueryio. #29383
Conversation
|
fixes: #29382 |
|
There seem to be very few existing tests in this package. I'll do some manual testing, though there is no significant logic to unit test (just that the added struct fields are copied over). I would appreciate any feedback! |
|
This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@beam.apache.org list. Thank you for your contributions. |
|
This pull request has been closed due to lack of activity. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time. |
1 participant
addresses #29382
Exposes options from the underlying go BigQuery sdk to the bigqueryio package, to allow for Parameters, DefaultProjectId and DefaultDatasetID.
This allows for queries that are not susceptible to SQL injection attacks. Most values can be replaced via parameters, but parameters cannot be used for identifiers. DefaultProjectId and DefaultDatasetID help by allowing unqualified table names to be used in the query, reducing the need for string interpolation on queries where table names are known ahead of time and tables share datasets.