Skip to content

Conversation

@stephaniewang526
Copy link
Contributor

@stephaniewang526 stephaniewang526 commented Aug 27, 2020

Expose BIGNUMERIC type in the client

cc: @MingyuZhong

@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Aug 27, 2020
@stephaniewang526 stephaniewang526 marked this pull request as draft August 27, 2020 19:18
@product-auto-label product-auto-label bot added the api: bigquery Issues related to the googleapis/java-bigquery API. label Aug 28, 2020
update IT testcase nit
@stephaniewang526 stephaniewang526 marked this pull request as ready for review August 28, 2020 16:23
@codecov
Copy link

codecov bot commented Aug 28, 2020

Codecov Report

Merging #703 (f021dde) into master (f59241b) will increase coverage by 0.67%.
The diff coverage is 100.00%.

Impacted file tree graph

@@ Coverage Diff @@ ## master #703 +/- ## ============================================ + Coverage 80.40% 81.07% +0.67%  + Complexity 1271 1237 -34  ============================================ Files 79 78 -1 Lines 6573 6336 -237 Branches 758 703 -55 ============================================ - Hits 5285 5137 -148  + Misses 893 833 -60  + Partials 395 366 -29 
Impacted Files Coverage Δ Complexity Δ
...a/com/google/cloud/bigquery/LegacySQLTypeName.java 95.34% <100.00%> (+0.22%) 7.00 <0.00> (ø)
...com/google/cloud/bigquery/QueryParameterValue.java 81.86% <100.00%> (+0.08%) 61.00 <1.00> (+1.00)
...com/google/cloud/bigquery/StandardSQLTypeName.java 100.00% <100.00%> (ø) 1.00 <0.00> (ø)
...google/cloud/bigquery/HivePartitioningOptions.java 83.78% <0.00%> (-1.64%) 11.00% <0.00%> (-2.00%)
...y/src/main/java/com/google/cloud/bigquery/Acl.java 71.42% <0.00%> (-0.58%) 13.00% <0.00%> (-1.00%)
...google/cloud/bigquery/ExternalTableDefinition.java 58.09% <0.00%> (-0.47%) 19.00% <0.00%> (-2.00%)
...m/google/cloud/bigquery/QueryJobConfiguration.java 90.08% <0.00%> (-0.27%) 66.00% <0.00%> (-2.00%)
...n/java/com/google/cloud/bigquery/BigQueryImpl.java 82.84% <0.00%> (-0.24%) 77.00% <0.00%> (-9.00%)
...main/java/com/google/cloud/bigquery/FieldList.java 100.00% <0.00%> (ø) 11.00% <0.00%> (ø%)
...va/com/google/cloud/bigquery/TimePartitioning.java 100.00% <0.00%> (ø) 7.00% <0.00%> (ø%)
... and 6 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f59241b...f517a6b. Read the comment docs.

Copy link

@MingyuZhong MingyuZhong left a comment

Choose a reason for hiding this comment

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

Thanks!

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

Cannot merge until BigQuery backend is ready.

@shollyman
Copy link
Contributor

Modulo any updates bringing this PR back in line with master, this should be good to merge now.

@stephaniewang526 stephaniewang526 requested a review from a team November 19, 2020 20:23
@stephaniewang526 stephaniewang526 removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Nov 19, 2020
@stephaniewang526 stephaniewang526 added the automerge Merge the pull request once unit tests and other checks pass. label Nov 19, 2020
@gcf-merge-on-green gcf-merge-on-green bot merged commit 718ec2a into googleapis:master Nov 19, 2020
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Nov 19, 2020
gcf-merge-on-green bot pushed a commit that referenced this pull request Nov 19, 2020
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.

4 participants