Skip to content

Conversation

@sduskis
Copy link
Contributor

@sduskis sduskis commented Apr 29, 2019

Adding a new MessagesBatch method:
private List<OutstandingBatch> add(OutstandingPublish outstandingPublish).

This method is responsible for adding a new user request in the form of a OutstandingPublish, and returning 0-2 OutstandingBatch, depending on the state of the current messages in flight, and the size of the new OutstandingPublish.

Adding a new `MessagesBatch` method:
`private List<OutstandingBatch> add(OutstandingPublish outstandingPublish)`.

This method is responsible for adding a new user request in the form of a `OutstandingPublish`, and returning 0-2 `OutstandingBatch`, depending on the state of the current messages in flight, and the size of the new `OutstandingPublish`.
@sduskis sduskis requested a review from a team as a code owner April 29, 2019 02:58
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Apr 29, 2019
@codecov
Copy link

codecov bot commented Apr 29, 2019

Codecov Report

Merging #5032 into master will decrease coverage by 0.33%.
The diff coverage is 75%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #5032      +/-   ##
============================================
- Coverage     50.33%   49.99%   -0.34%     
+ Complexity    23676    23672       -4     
============================================
  Files          2238     2238              
  Lines        226057   226006      -51     
  Branches      24961    24194     -767     
============================================
- Hits         113775   112998     -777     
+ Misses       103678   103665      -13     
- Partials       8604     9343     +739
Impacted Files Coverage Δ Complexity Δ
...ain/java/com/google/cloud/pubsub/v1/Publisher.java 84.78% <75%> (-0.75%) 24 <0> (-4)
...va/com/google/cloud/compute/v1/InstanceClient.java 48.95% <0%> (-6.46%) 139% <0%> (ø)
...ava/com/google/cloud/compute/v1/ProjectClient.java 51.14% <0%> (-5.94%) 55% <0%> (ø)
...ava/com/google/cloud/compute/v1/NetworkClient.java 49.63% <0%> (-5.84%) 35% <0%> (ø)
...ava/com/google/cloud/compute/v1/LicenseClient.java 50.41% <0%> (-5.79%) 31% <0%> (ø)
.../com/google/cloud/compute/v1/RegionDiskClient.java 48.92% <0%> (-5.76%) 35% <0%> (ø)
...va/com/google/cloud/compute/v1/SnapshotClient.java 50% <0%> (-5.74%) 31% <0%> (ø)
.../java/com/google/cloud/compute/v1/ImageClient.java 49.04% <0%> (-5.74%) 43% <0%> (ø)
.../google/cloud/compute/v1/TargetSslProxyClient.java 48.57% <0%> (-5.72%) 35% <0%> (ø)
.../google/cloud/compute/v1/SecurityPolicyClient.java 48.1% <0%> (-5.7%) 39% <0%> (ø)
... and 142 more

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 ac0084f...bb2af62. Read the comment docs.

@sduskis sduskis merged commit 61b9e4f into googleapis:master Apr 30, 2019
@sduskis sduskis deleted the refactor_publish_0425 branch April 30, 2019 15:44
@JesseLovelace JesseLovelace mentioned this pull request May 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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