Skip to content

Adjusted error handling based on Googler feedback - #693

Merged
theacodes merged 1 commit into
masterfrom
bigquery-loadgcsdata-errorhandling
Nov 30, 2016
Merged

Adjusted error handling based on Googler feedback#693
theacodes merged 1 commit into
masterfrom
bigquery-loadgcsdata-errorhandling

Conversation

@ryanmats

Copy link
Copy Markdown
Contributor

Adjusted error handling for BigQuery sample based on Googler feedback. Job.errors gives you the actual error(s), unlike job.error_result.

Bug #33164873

@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Nov 30, 2016
@theacodes

Copy link
Copy Markdown
Contributor

@dpebot can you merge when travis passes?

@dpebot

Copy link
Copy Markdown
Collaborator

Okay! I'll merge when all statuses are green.

@dpebotdpebot added the automerge Merge the pull request once unit tests and other checks pass. label Nov 30, 2016
@theacodes

Copy link
Copy Markdown
Contributor

Failures seem to be due to bigquery being flaky, merging.

@theacodes
theacodes merged commit 92ebe4f into masterNov 30, 2016
@theacodes
theacodes deleted the bigquery-loadgcsdata-errorhandling branch November 30, 2016 21:18
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automergeMerge the pull request once unit tests and other checks pass.cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@ryanmats@theacodes@dpebot@googlebot