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 new scalar statistics properties to 'QueryJob' #3800

Merged
merged 7 commits into from
Sep 7, 2017
Merged

Add new scalar statistics properties to 'QueryJob' #3800

merged 7 commits into from
Sep 7, 2017

Conversation

tseaver
Copy link
Contributor

@tseaver tseaver commented Aug 12, 2017

  • total_bytes_processed
  • total_bytes_billed
  • billing_tier
  • cache_hit
  • num_dml_affected_rows
  • statement_type

Uses #3799 as a base.

Cherry-picked / fixed up from PR #3721.

@tseaver tseaver added the api: bigquery Issues related to the BigQuery API. label Aug 12, 2017
@tseaver tseaver requested review from tswast and dhermes August 12, 2017 20:10
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Aug 12, 2017
@tseaver tseaver changed the base branch from master to bigquery-b2 August 12, 2017 20:42
:returns: total bytes processed by the job, or None if job is not
yet complete.
"""
return self._job_statistics().get('totalBytesProcessed')

This comment was marked as spam.

:returns: total bytes processed by the job, or None if job is not
yet complete.
"""
return self._job_statistics().get('totalBytesBilled')

This comment was marked as spam.

:returns: number of DML rows affectd by the job, or None if job is not
yet complete.
"""
return self._job_statistics().get('numDmlAffectedRows')

This comment was marked as spam.

@tseaver
Copy link
Contributor Author

tseaver commented Sep 7, 2017

@tswast I have rebased this after merging #3799 to the bigquery-b2 branch, and added the requested integer conversions. PTAL

@tseaver
Copy link
Contributor Author

tseaver commented Sep 7, 2017

BQ tests all pass: the failure is in the docs build (now fixed on master, but not the bigquery-b2 branch).

@tseaver tseaver merged commit 06ac65e into googleapis:bigquery-b2 Sep 7, 2017
@tseaver tseaver deleted the bigquery-b2-queryjob-scalar_stats_properties branch September 7, 2017 21:46
tseaver added a commit that referenced this pull request Sep 11, 2017
* Add 'QueryJob.total_bytes_processed' property.
* Add 'QueryJob.total_bytes_billed' property.
* Add 'QueryJob.billing_tier' property.
* Add 'QueryJob.cache_hit' property.
* Add 'QueryJob.num_dml_affected_rows' property.
* Add 'QueryJob.statement_type' property.
tswast pushed a commit that referenced this pull request Sep 25, 2017
* Add 'QueryJob.total_bytes_processed' property.
* Add 'QueryJob.total_bytes_billed' property.
* Add 'QueryJob.billing_tier' property.
* Add 'QueryJob.cache_hit' property.
* Add 'QueryJob.num_dml_affected_rows' property.
* Add 'QueryJob.statement_type' property.
@sschatts
Copy link

@tseaver Do you have a planned date this will be released?

@tseaver
Copy link
Contributor Author

tseaver commented Oct 10, 2017

@sschatts There is a fairly intrusive bigquery API refactoring going on at the moment: I believe we are hoping to get a "beta 2" release cut within the next couple of weeks.

@sschatts
Copy link

@tseaver Thank you for the update.

tswast pushed a commit that referenced this pull request Oct 12, 2017
* Add 'QueryJob.total_bytes_processed' property.
* Add 'QueryJob.total_bytes_billed' property.
* Add 'QueryJob.billing_tier' property.
* Add 'QueryJob.cache_hit' property.
* Add 'QueryJob.num_dml_affected_rows' property.
* Add 'QueryJob.statement_type' property.
tswast pushed a commit that referenced this pull request Oct 16, 2017
* Add 'QueryJob.total_bytes_processed' property.
* Add 'QueryJob.total_bytes_billed' property.
* Add 'QueryJob.billing_tier' property.
* Add 'QueryJob.cache_hit' property.
* Add 'QueryJob.num_dml_affected_rows' property.
* Add 'QueryJob.statement_type' property.
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 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.

4 participants