Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add JsonStreamWriter #475

Merged
merged 41 commits into from
Aug 7, 2020

Conversation

allenc3
Copy link
Contributor

@allenc3 allenc3 commented Aug 7, 2020

Changed initial commit message to be feat: add JsonStreamWriter

@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Aug 7, 2020
@codecov
Copy link

codecov bot commented Aug 7, 2020

Codecov Report

Merging #475 into master will increase coverage by 0.29%.
The diff coverage is 79.08%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #475      +/-   ##
============================================
+ Coverage     78.00%   78.30%   +0.29%     
- Complexity      624      642      +18     
============================================
  Files            56       58       +2     
  Lines          3551     3687     +136     
  Branches        235      245      +10     
============================================
+ Hits           2770     2887     +117     
- Misses          675      687      +12     
- Partials        106      113       +7     
Impacted Files Coverage Δ Complexity Δ
.../cloud/bigquery/storage/v1alpha2/DirectWriter.java 80.64% <ø> (ø) 5.00 <0.00> (ø)
...igquery/storage/v1alpha2/ProtoSchemaConverter.java 96.22% <ø> (+14.40%) 10.00 <0.00> (+1.00)
...ud/bigquery/storage/v1alpha2/JsonStreamWriter.java 77.19% <77.19%> (ø) 12.00 <12.00> (?)
.../cloud/bigquery/storage/v1alpha2/StreamWriter.java 82.22% <84.37%> (+0.69%) 35.00 <7.00> (+1.00)
...query/storage/v1alpha2/OnSchemaUpdateRunnable.java 85.71% <85.71%> (ø) 4.00 <4.00> (?)

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 5b6cadd...ce3459a. Read the comment docs.

@stephaniewang526 stephaniewang526 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Aug 7, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Aug 7, 2020
@stephaniewang526 stephaniewang526 added the automerge Merge the pull request once unit tests and other checks pass. label Aug 7, 2020
@gcf-merge-on-green gcf-merge-on-green bot merged commit 341bf99 into googleapis:master Aug 7, 2020
gcf-merge-on-green bot pushed a commit that referenced this pull request Aug 7, 2020
🤖 I have created a release \*beep\* \*boop\* 
---
## [1.4.0](https://www.github.com/googleapis/java-bigquerystorage/compare/v1.3.1...v1.4.0) (2020-08-07)


### Features

* add JsonStreamWriter ([#475](https://www.github.com/googleapis/java-bigquerystorage/issues/475)) ([341bf99](https://www.github.com/googleapis/java-bigquerystorage/commit/341bf99d47d788ebabaa1ce775d1a4f527ee46b0))


### Bug Fixes

* enum value conflict in generated ProtoSchema descriptor. ([#469](https://www.github.com/googleapis/java-bigquerystorage/issues/469)) ([3e1382f](https://www.github.com/googleapis/java-bigquerystorage/commit/3e1382f247de5e6ee8727130280e34fa01d3c088)), closes [#464](https://www.github.com/googleapis/java-bigquerystorage/issues/464)


### Dependencies

* update dependency com.fasterxml.jackson.core:jackson-core to v2.11.2 ([#461](https://www.github.com/googleapis/java-bigquerystorage/issues/461)) ([36b368a](https://www.github.com/googleapis/java-bigquerystorage/commit/36b368a14bb7d45f92af23bc0dd6fe08ad79b085))
* update dependency com.google.cloud:google-cloud-bigquery to v1.116.9 ([#473](https://www.github.com/googleapis/java-bigquerystorage/issues/473)) ([b4ba055](https://www.github.com/googleapis/java-bigquerystorage/commit/b4ba055a809dbe414e8e19d377b8257d0b5446d6))
* update dependency com.google.cloud:google-cloud-shared-dependencies to v0.8.5 ([#467](https://www.github.com/googleapis/java-bigquerystorage/issues/467)) ([35c65d4](https://www.github.com/googleapis/java-bigquerystorage/commit/35c65d466d44d01e6f7626ef7cf1aa0e9f3ff0f3))
---


This PR was generated with [Release Please](https://github.com/googleapis/release-please).
shollyman pushed a commit to shollyman/java-bigquerystorage that referenced this pull request Apr 7, 2021
* Change triggered by none of the following:
This git repo (https://github.com/googleapis/java-bigquerystorage.git)
Git repo https://github.com/googleapis/googleapis.git
Git repo https://github.com/googleapis/synthtool.git

* build(java): fix retry_with_backoff when -e option set (googleapis#475)

googleapis/synthtool@bd69a2a
commit bd69a2aa7b70875f3c988e269706b22fefbef40e
Author: Jeff Ching <chingor@google.com>
Date:   Wed Apr 8 14:01:08 2020 -0700

    build(java): fix retry_with_backoff when -e option set (googleapis#475)

* build(java): fix nightly integration test config to run integrations (googleapis#465)

googleapis/synthtool@c7e0e51
commit c7e0e517d7f46f77bebd27da2e5afcaa6eee7e25
Author: Jeff Ching <chingor@google.com>
Date:   Wed Apr 8 14:06:04 2020 -0700

    build(java): fix nightly integration test config to run integrations (googleapis#465)

    This was only running the units.

* chore(java): release-please only updates non maven versions in README (googleapis#476)

googleapis/synthtool@7f8e62a
commit 7f8e62aa3edd225f76347a16f92e400661fdfb52
Author: Jeff Ching <chingor@google.com>
Date:   Wed Apr 8 14:14:25 2020 -0700

    chore(java): release-please only updates non maven versions in README (googleapis#476)

    Prevent release-please and synthtool from fighting over the released library version. Synthtool updates the install snippets from the samples pom.xml files so the bots fight if they are temporarily out of sync after a release.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge the pull request once unit tests and other checks pass. 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