Skip to content

Conversation

@tswast
Copy link
Contributor

@tswast tswast commented Feb 7, 2018

Also, removes assert for state == 'RUNNING' from snippets, as this is flaky for small jobs that complete very quickly.

The key used is one that I've created which is accessible to all users. This means anyone can still run the system tests.

Based on #4845

@tswast tswast added api: bigquery Issues related to the BigQuery API. do not merge Indicates a pull request not ready for merge, due to either quality or timing. labels Feb 7, 2018
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 7, 2018
@tswast tswast force-pushed the bq-cmek-snippets branch 2 times, most recently from b82f1fb to b909b6d Compare February 8, 2018 23:46
Addition samples for BigQuery CMEK:

- Create table
- Update table
- Query destination table
- Copy table
- Load table

Also, removes assert for state == 'RUNNING' from snippets, as this is flaky for small jobs that complete very quickly.
@tswast tswast requested a review from theacodes February 8, 2018 23:51
@tswast
Copy link
Contributor Author

tswast commented Feb 8, 2018

/cc @vitaliili

@tswast tswast removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Feb 8, 2018
@tswast tswast changed the title BigQuery: add code snippet for CMEK table loading. [DO NOT MERGE] BigQuery: add code snippet for CMEK table loading. Feb 8, 2018
@tswast tswast merged commit 176b0ae into googleapis:master Feb 9, 2018
@tswast tswast deleted the bq-cmek-snippets branch February 9, 2018 00:26
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