Skip to content

Conversation

plamut
Copy link
Contributor

@plamut plamut commented Jan 14, 2021

Fixes #469.

This PR fixes reading the labels on Job instances and adds a system test that would have caught this bug.

PR checklist:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

@plamut plamut requested review from a team and steffnay and removed request for a team January 14, 2021 10:43
@product-auto-label product-auto-label bot added the api: bigquery Issues related to the googleapis/python-bigquery API. label Jan 14, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Jan 14, 2021
Copy link
Contributor

@tswast tswast left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Can confirm in https://cloud.google.com/bigquery/docs/reference/rest/v2/Job#JobConfiguration that this is how it's supposed to be (labels are a property of configuration)

@tswast tswast merged commit 80944f0 into googleapis:master Jan 14, 2021
@plamut plamut deleted the iss-469 branch January 14, 2021 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: bigquery Issues related to the googleapis/python-bigquery API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Reading the labels set on a job fails

2 participants