Skip to content

Conversation

@stephaniewang526
Copy link
Contributor

@stephaniewang526 stephaniewang526 commented Jun 2, 2021

Towards b/188414733
Implemented in LoadJobConfiguration and ExternalTableDefinition

cc @MingyuZhong

Towards b/188414733
Implemented in LoadJobConfiguration
@stephaniewang526 stephaniewang526 requested review from a team and prash-mi June 2, 2021 18:17
@product-auto-label product-auto-label bot added the api: bigquery Issues related to the googleapis/java-bigquery API. label Jun 2, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Jun 2, 2021
@stephaniewang526 stephaniewang526 removed the request for review from prash-mi June 2, 2021 18:18
@stephaniewang526 stephaniewang526 marked this pull request as draft June 2, 2021 18:18
@stephaniewang526 stephaniewang526 marked this pull request as ready for review June 7, 2021 16:23
@stephaniewang526 stephaniewang526 requested a review from a team as a code owner June 7, 2021 16:23
@stephaniewang526 stephaniewang526 changed the title feat: add support for DecimalTargetType feat: add support for DecimalTargetTypes Jun 7, 2021
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.

Nice, I like the exercise of the precision/scale validation in the integration test.

@stephaniewang526 stephaniewang526 added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jun 8, 2021
@stephaniewang526
Copy link
Contributor Author

Will not merge until the UI change reaches golden.

@stephaniewang526 stephaniewang526 removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jun 8, 2021
@stephaniewang526 stephaniewang526 merged commit ba528df into googleapis:master Jun 8, 2021
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 googleapis/java-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