Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add configure_job method to BigQuery job tasks #3098

Merged
merged 1 commit into from
Aug 23, 2021

Conversation

sisidra
Copy link
Contributor

@sisidra sisidra commented Aug 23, 2021

Description

This allows to configure job with any parameters, not only the ones exposed via properties.

Motivation and Context

Currently some job parameters are not exposed via default job parameters.

Given that BigQuery API can change over time and add more parameters,
I think that adding configuration method to give full control over job configuration
to the user removes friction and custom code hacks users need to implement.

Have you tested this? If so, how?

  • I have included unit tests
  • I ran the code for BigQueryRunQueryTask implementation

This allows to configure job with any parameters, not only the ones exposed via properties.
@sisidra sisidra requested review from dlstadther, Tarrasch and a team as code owners August 23, 2021 11:19
@dlstadther dlstadther merged commit 6b42426 into spotify:master Aug 23, 2021
@sisidra sisidra deleted the configure-bigquery-jobs branch August 24, 2021 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants