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

fix: Accept null json values in JsonToProtoMessage converter#1288

Merged
stephaniewang526 merged 3 commits intogoogleapis:masterfrom
yirutang:nullvalue
Sep 3, 2021
Merged

fix: Accept null json values in JsonToProtoMessage converter#1288
stephaniewang526 merged 3 commits intogoogleapis:masterfrom
yirutang:nullvalue

Conversation

@yirutang
Copy link
Copy Markdown
Contributor

@yirutang yirutang commented Sep 3, 2021

Fixes #1258 ☕️

@yirutang yirutang requested review from a team and tswast September 3, 2021 00:10
@product-auto-label product-auto-label bot added the api: bigquerystorage Issues related to the googleapis/java-bigquerystorage API. label Sep 3, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Sep 3, 2021
@stephaniewang526 stephaniewang526 added owlbot:run Add this label to trigger the Owlbot post processor. automerge Merge the pull request once unit tests and other checks pass. labels Sep 3, 2021
@yirutang yirutang requested a review from a team September 3, 2021 01:18
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Sep 3, 2021
@stephaniewang526
Copy link
Copy Markdown
Contributor

could you see if the test case needs to be updated?

Error:  Errors: 
Error:    StreamWriterTest.testFlowControlBehaviorBlockAbortOnShutdown:984 » NullPointer

Full trace

@gcf-merge-on-green
Copy link
Copy Markdown

Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot.

@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Sep 3, 2021
@stephaniewang526 stephaniewang526 added the owlbot:run Add this label to trigger the Owlbot post processor. label Sep 3, 2021
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Sep 3, 2021
@stephaniewang526 stephaniewang526 added the automerge Merge the pull request once unit tests and other checks pass. label Sep 3, 2021
@stephaniewang526 stephaniewang526 merged commit fb515ab into googleapis:master Sep 3, 2021
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Sep 3, 2021
gcf-owl-bot bot added a commit that referenced this pull request Nov 29, 2021
* deps: upgrade google-java-format to 1.11.0

Fixes googleapis/synthtool#1287

google-java-format 1.11.0 is the version used by fmt-maven-plugin 2.12
(latest as of November 2021).

* ci: owlbot to use Java 11

* formatter version through the java-shared-config

* updating container_test.yaml
Source-Link: googleapis/synthtool@fdf20ea
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-java:latest@sha256:55d5d3c72e1c065f018e423e122a22fb55c8daa36f6f17f540774779ed659ab9
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: bigquerystorage Issues related to the googleapis/java-bigquerystorage API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

JsonToProtoMessage.fillField can not handle null values

2 participants