Skip to content

Conversation

@yirutang
Copy link
Contributor

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> ☕️

@yirutang yirutang requested a review from a team October 22, 2020 01:04
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Oct 22, 2020
@codecov
Copy link

codecov bot commented Oct 22, 2020

Codecov Report

Merging #627 into master will increase coverage by 0.09%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #627      +/-   ##
============================================
+ Coverage     78.63%   78.73%   +0.09%     
+ Complexity      663      660       -3     
============================================
  Files            59       59              
  Lines          3782     3766      -16     
  Branches        253      249       -4     
============================================
- Hits           2974     2965       -9     
+ Misses          692      689       -3     
+ Partials        116      112       -4     
Impacted Files Coverage Δ Complexity Δ
...ud/bigquery/storage/v1alpha2/JsonStreamWriter.java 77.19% <ø> (-0.20%) 12.00 <0.00> (-1.00)
.../cloud/bigquery/storage/v1alpha2/StreamWriter.java 84.31% <ø> (+0.27%) 37.00 <0.00> (-2.00) ⬆️
...oud/bigquery/storage/v1alpha2/JsonWriterCache.java 91.07% <100.00%> (+3.14%) 12.00 <0.00> (ø)
...e/cloud/bigquery/storage/v1alpha2/WriterCache.java 94.25% <100.00%> (+2.11%) 14.00 <0.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 6347c13...571dae2. Read the comment docs.

@yirutang yirutang requested a review from a team as a code owner October 22, 2020 01:13
@product-auto-label product-auto-label bot added the api: bigquerystorage Issues related to the googleapis/java-bigquerystorage API. label Oct 22, 2020
@stephaniewang526 stephaniewang526 merged commit 2ae69b6 into googleapis:master Oct 22, 2020
gcf-merge-on-green bot pushed a commit that referenced this pull request Oct 27, 2020
🤖 I have created a release \*beep\* \*boop\* 
---
## [1.6.0](https://www.github.com/googleapis/java-bigquerystorage/compare/v1.5.6...v1.6.0) (2020-10-27)


### Features

* bigquery Write API V1Beta2 public interface. ([#637](https://www.github.com/googleapis/java-bigquerystorage/issues/637)) ([bb21e7b](https://www.github.com/googleapis/java-bigquerystorage/commit/bb21e7bc2b206e7720855d032889f4bc45121f9a))


### Bug Fixes

* remove stream ttl in client library, since there is no very clear TTL defined. ([#627](https://www.github.com/googleapis/java-bigquerystorage/issues/627)) ([2ae69b6](https://www.github.com/googleapis/java-bigquerystorage/commit/2ae69b640adc48b79f0aab71c215eb3ef055a34c))


### Dependencies

* update dependency com.google.cloud:google-cloud-bigquery to v1.123.1 ([#632](https://www.github.com/googleapis/java-bigquerystorage/issues/632)) ([7b5fc9b](https://www.github.com/googleapis/java-bigquerystorage/commit/7b5fc9b917035ca3b419b71abb0f9d6a4ca59950))
* update dependency com.google.cloud:google-cloud-bigquery to v1.123.2 ([#633](https://www.github.com/googleapis/java-bigquerystorage/issues/633)) ([bf6260e](https://www.github.com/googleapis/java-bigquerystorage/commit/bf6260ebe01d040e42f3eb4b3b8262054f9316bf))
---


This PR was generated with [Release Please](https://github.com/googleapis/release-please).
@yirutang yirutang deleted the remove-ttl branch December 10, 2020 23:28
shubhwip pushed a commit to shubhwip/java-bigquerystorage that referenced this pull request Oct 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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.

2 participants