Skip to content
This repository has been archived by the owner on Sep 26, 2023. It is now read-only.

Expose Batcher#sendOutstanding to provide async batch flushing #786

Merged
merged 1 commit into from Sep 9, 2019

Conversation

rahulKQL
Copy link
Contributor

@rahulKQL rahulKQL commented Sep 5, 2019

Now Batcher supports asynchronously batching.

Note: This is needed to support HBase client's BulkMutation#sendUnsent through GCJ client.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 5, 2019
@rahulKQL rahulKQL requested a review from kolea2 September 5, 2019 13:16
@codecov
Copy link

codecov bot commented Sep 5, 2019

Codecov Report

Merging #786 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #786   +/-   ##
=========================================
  Coverage     78.39%   78.39%           
  Complexity     1106     1106           
=========================================
  Files           198      198           
  Lines          4887     4887           
  Branches        385      385           
=========================================
  Hits           3831     3831           
  Misses          887      887           
  Partials        169      169
Impacted Files Coverage Δ Complexity Δ
.../java/com/google/api/gax/batching/BatcherImpl.java 97.9% <100%> (ø) 15 <1> (ø) ⬇️

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 a4aec0f...4e5be5b. Read the comment docs.

Copy link
Contributor

@igorbernstein2 igorbernstein2 left a comment

Choose a reason for hiding this comment

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

Lgtm

@rahulKQL rahulKQL merged commit 7637f9c into googleapis:master Sep 9, 2019
This was referenced Sep 27, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants