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

feat(bigquery): add range partitioning to tables, load jobs, and query jobs #9477

Merged
merged 7 commits into from
Oct 18, 2019

Conversation

tswast
Copy link
Contributor

@tswast tswast commented Oct 15, 2019

These classes and properties add support for the integer range
partitioning feature. These offer more flexibility in partitioning
options than time-based partitioning.

Note: integer range partitioning is currently only available under whitelist.

…y jobs

These classes and properties add support for the integer range
partitioning feature. These offer more flexibility in partitioning
options than time-based partitioning.
@tswast tswast added the api: bigquery Issues related to the BigQuery API. label Oct 15, 2019
@tswast tswast requested a review from a team October 15, 2019 21:03
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 15, 2019
@tswast
Copy link
Contributor Author

tswast commented Oct 15, 2019

FYI: I've started a draft PR at #9478 with code samples (doubling as system tests). I figure we can get this PR committed and released and follow-up with samples once the feature is no longer restricted access.

@tswast tswast requested a review from shollyman October 16, 2019 15:47
@tswast tswast added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Oct 16, 2019
Copy link
Contributor

@shollyman shollyman left a comment

Choose a reason for hiding this comment

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

Looks good. Will keep an eye out for the system test PR as well.

@@ -27,7 +27,9 @@
from google.cloud.bigquery.dataset import Dataset
from google.cloud.bigquery.dataset import DatasetListItem
from google.cloud.bigquery.dataset import DatasetReference
from google.cloud.bigquery.encryption_configuration import EncryptionConfiguration
Copy link
Contributor

Choose a reason for hiding this comment

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

cleanup from another PR?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, recently added for models encryption but I forgot to ask for the imports to be alphabetized.

@tswast tswast merged commit ecb2162 into googleapis:master Oct 18, 2019
@tswast tswast removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Oct 18, 2019
@tswast tswast deleted the bq-range-partitioning branch October 18, 2019 16:40
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.

3 participants