Skip to content

BigQuery: Add 'QueryJob.estimated_bytes_processed' property - #5655

Merged
tseaver merged 19 commits into
googleapis:masterfrom
tseaver:5646-bigquery-job-estimated_bytes_processed
Jul 19, 2018
Merged

BigQuery: Add 'QueryJob.estimated_bytes_processed' property#5655
tseaver merged 19 commits into
googleapis:masterfrom
tseaver:5646-bigquery-job-estimated_bytes_processed

Conversation

@tseaver

@tseavertseaver commented Jul 19, 2018

Copy link
Copy Markdown
Contributor

Shares the unit test backfill stuff (515a0f5) with PR #5654.

Closes#5646.

tseaver added 19 commits July 19, 2018 11:19
@tseaver
Avoid importing module-/package-under-test at testcase module scope.
770b560
@tseaver
Add explict unit tests for 'job._JobReference'.
d8859b4
@tseaver
Add explicit unit tests for '_AsyncJob' ctor and read-only properties.
8c48c79
@tseaver
Add explicit unit tests for '_AsyncJob._set_properties' and helpers.
64453b4
@tseaver
Add explict unit tests for '_AsyncJob._get_resource_config'.
0b6f7d7
@tseaver
Note '_AsyncJob._build_resource' explicitly as an abstract method.
7dceb43
Update docstrings to note use by '_begin', rather than 'begin'.
@tseaver
Add explicit unit tests for '_AsyncJob._begin'.
4ba5dc9
@tseaver
Add explicit unit tests for '_AsyncJob.exists'.
ad31f21
@tseaver
Add explicit unit tests for '_AsyncJob.reload'.
8628ca5
@tseaver
Add explicit unit tests for '_AsyncJob.cancel'.
d47e7ff
@tseaver
Add explicit unit tests for '_AsyncJob._set_future_result'.
1f8fcc3
@tseaver
Add explicit unit tests for '_AsyncJob.done'.
af94667
@tseaver
Add explicit unit tests for '_AsyncJob.resuit'.
8f63531
@tseaver
Add explicit unit tests for '_AsyncJob.cancelled'.
16f4d6a
@tseaver
Add explicit unit test for '_JobConfig' ctor.
752c74a
@tseaver
Add explicit unit tests for '_JobConfig._{get,set}_sub_prop'.
9003cf0
@tseaver
Add explicit unit test for '_JobConfig.to_api_repr'.
3ee7d8c
@tseaver
Note why '_JobConfig.from_api_repr' is not tested directly.
515a0f5
@tseaver
Add 'QueryJob.estimated_bytes_proceesed' property.
d7eeccb
Closes#5646.
@tseavertseaver added the api: bigquery Issues related to the BigQuery API. label Jul 19, 2018
@tseaver
tseaver requested a review from tswastJuly 19, 2018 15:36
@tseaver
tseaver requested a review from theacodes as a code ownerJuly 19, 2018 15:36
@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 19, 2018
tswast
tswast approved these changes Jul 19, 2018
View reviewed changes

@tswasttswast left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks!

@tseaver
tseaver merged commit ce951de into googleapis:masterJul 19, 2018
@tseaver
tseaver deleted the 5646-bigquery-job-estimated_bytes_processed branch July 19, 2018 18:34
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tswasttswasttswast approved these changes
@theacodestheacodesAwaiting requested review from theacodes

Assignees

No one assigned

Labels

api: bigqueryIssues related to the BigQuery API.cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tseaver@tswast@googlebot