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 review from a team and tswast December 14, 2020 21:00
@product-auto-label product-auto-label bot added the api: bigquerystorage Issues related to the googleapis/java-bigquerystorage API. label Dec 14, 2020
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Dec 14, 2020
ApiFuture<AppendRowsResponse> response =
jsonStreamWriter.append(row, -1, /* allowUnknownFields */ false);
assertEquals(0, response.get().getOffset());
jsonStreamWriter.append(row, -1, /* allowUnknownFields */ false);
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
jsonStreamWriter.append(row, -1, /* allowUnknownFields */ false);
jsonStreamWriter.append(row, -1, /* allowUnknownFields */ false);


ApiFuture<AppendRowsResponse> response =
jsonStreamWriter.append(row, -1, /* allowUnknownFields */ false);

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change

@codecov
Copy link

codecov bot commented Dec 14, 2020

Codecov Report

Merging #731 (d3518aa) into master (a1b1c8e) will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #731   +/-   ##
=========================================
  Coverage     80.25%   80.25%           
  Complexity      957      957           
=========================================
  Files            73       73           
  Lines          5324     5319    -5     
  Branches        403      401    -2     
=========================================
- Hits           4273     4269    -4     
  Misses          880      880           
+ Partials        171      170    -1     
Impacted Files Coverage Δ Complexity Δ
...e/cloud/bigquery/storage/v1beta2/StreamWriter.java 83.78% <ø> (+0.04%) 37.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 a1b1c8e...d3518aa. Read the comment docs.

@yirutang yirutang changed the title fix: Disable Breaking change related code site. fix: Disable Breaking change related code site Dec 15, 2020
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.

3 participants