Skip to content

doc(bigquery): add table create sample using integer range partitioning - #9478

Merged
plamut merged 4 commits into
googleapis:masterfrom
tswast:bq-range-partitioning-samples
Nov 6, 2019
Merged

doc(bigquery): add table create sample using integer range partitioning#9478
plamut merged 4 commits into
googleapis:masterfrom
tswast:bq-range-partitioning-samples

Conversation

@tswast

@tswasttswast commented Oct 15, 2019

Copy link
Copy Markdown
Contributor

This code sample doubles as a system test for integer range partitioning features. Follow-up to #9477

@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 15, 2019
This code sample doubles as a system test for integer range partitioning
features.
@tswast
tswastforce-pushed the bq-range-partitioning-samples branch from 846b60c to e1f6921CompareOctober 18, 2019 16:47
@tswasttswast changed the title doc(bigquery): add code samples for integer range partitioningdoc(bigquery): add table create sample using integer range partitioningOct 18, 2019
@tswast
tswast marked this pull request as ready for review October 18, 2019 16:47
@tswast
tswast requested review from a team and shollymanOctober 18, 2019 16:47
@plamutplamut added type: docs Improvement to the documentation for an API. api: bigquery Issues related to the BigQuery API. labels Oct 22, 2019

@plamutplamut 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.

Looks good, please just blacken the changes (the lint check fails).

I noticed that table range partitioning info is not presented anywhere in the GCP control panel, but that's not a client library issue.

@tswast
tswast requested a review from plamutOctober 24, 2019 22:44

@plamutplamut 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.

As discussed offline - even if not necessary, it is a good idea to include the sample in the Sphinx docs, too.

@tswast
tswast requested a review from plamutOctober 25, 2019 22:38
@plamut

plamut commented Nov 4, 2019

Copy link
Copy Markdown
Contributor

@shollyman Is there anything that you might want to add here before merging?

Thanks!

Update: Seems like no, merging then. :)

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: bigqueryIssues related to the BigQuery API.cla: yesThis human has signed the Contributor License Agreement.type: docsImprovement to the documentation for an API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@tswast@plamut@googlebot