Skip to content
This repository was archived by the owner on Mar 23, 2026. It is now read-only.

feat: add support for DecimalTargetTypes - #1345

Merged
stephaniewang526 merged 5 commits into
googleapis:masterfrom
stephaniewang526:decimalTarget
Jun 8, 2021
Merged

feat: add support for DecimalTargetTypes#1345
stephaniewang526 merged 5 commits into
googleapis:masterfrom
stephaniewang526:decimalTarget

Conversation

@stephaniewang526

@stephaniewang526 stephaniewang526 commented Jun 2, 2021

Copy link
Copy Markdown
Contributor

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 review from a team and shollyman June 7, 2021 16:23
@stephaniewang526 stephaniewang526 changed the title feat: add support for DecimalTargetType feat: add support for DecimalTargetTypes Jun 7, 2021
Comment thread google-cloud-bigquery/clirr-ignored-differences.xml

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

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
Copy Markdown
Contributor Author

Will not merge until the UI change reaches golden.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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