Skip to content
New issue

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

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

Already on GitHub? # to your account

Pub/Sub: more refactoring #5044

Merged
merged 3 commits into from
May 1, 2019
Merged

Conversation

sduskis
Copy link
Contributor

@sduskis sduskis commented May 1, 2019

  • Using Preconditions
  • Adding onSuccess and onFailure to OutstandingPublish

- Using `Preconditions`
- Adding `onSuccess` and `onFailure` to `OutstandingPublish`
@sduskis sduskis requested a review from a team as a code owner May 1, 2019 14:18
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label May 1, 2019
@chingor13
Copy link
Contributor

Looks like you need to run the code formatter again.

@codecov
Copy link

codecov bot commented May 1, 2019

Codecov Report

Merging #5044 into master will decrease coverage by 0.64%.
The diff coverage is 70.58%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #5044      +/-   ##
============================================
- Coverage     50.35%   49.71%   -0.65%     
+ Complexity    23685    22317    -1368     
============================================
  Files          2242     2242              
  Lines        226183   221462    -4721     
  Branches      24961    24081     -880     
============================================
- Hits         113888   110091    -3797     
+ Misses       103693   102912     -781     
+ Partials       8602     8459     -143
Impacted Files Coverage Δ Complexity Δ
...ain/java/com/google/cloud/pubsub/v1/Publisher.java 88.98% <70.58%> (+2.34%) 23 <6> (ø) ⬇️
...in/java/com/google/cloud/storage/StorageBatch.java 88% <0%> (-4%) 13% <0%> (ø)
...e/cloud/vision/v1p4beta1/ImageAnnotatorClient.java 58.69% <0%> (-3.31%) 15% <0%> (-4%)
...m/google/cloud/vision/v1/ImageAnnotatorClient.java 47.05% <0%> (-2.95%) 9% <0%> (-2%)
...e/cloud/vision/v1p3beta1/ImageAnnotatorClient.java 47.05% <0%> (-2.95%) 9% <0%> (-2%)
...e/cloud/vision/v1p2beta1/ImageAnnotatorClient.java 47.05% <0%> (-2.95%) 9% <0%> (-2%)
.../java/com/google/cloud/speech/v1/SpeechClient.java 48.57% <0%> (-2.78%) 10% <0%> (-2%)
...om/google/cloud/speech/v1p1beta1/SpeechClient.java 48.57% <0%> (-2.78%) 10% <0%> (-2%)
...d/webrisk/v1beta1/WebRiskServiceV1Beta1Client.java 63.41% <0%> (-2.5%) 12% <0%> (-3%)
...ogle/cloud/texttospeech/v1/TextToSpeechClient.java 55.88% <0%> (-2.46%) 9% <0%> (-2%)
... and 543 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 860fe58...ceb2b47. Read the comment docs.

@sduskis sduskis merged commit 67668c1 into googleapis:master May 1, 2019
@sduskis sduskis deleted the preconditions_ps branch May 1, 2019 17:18
@JesseLovelace JesseLovelace mentioned this pull request May 1, 2019
# for free to join this conversation on GitHub. Already have an account? # 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