Skip to content

fix: if a flow control setting is not provided use zero #292

New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom
Merged
hannahrogers-google merged 31 commits into from
Jul 24, 2020
Merged

fix: if a flow control setting is not provided use zero #292

hannahrogers-google merged 31 commits into from
Jul 24, 2020

Conversation

@hannahrogers-google
Copy link
Contributor

@hannahrogers-google hannahrogers-google commented Jul 23, 2020

Fixes #291 ☕️

@hannahrogers-google
Modifying Publish example in README to match other examples given, and
19dd129 
fix issue #6784
@hannahrogers-google
fix: Modifying Publish example in README to match other examples, and
4158529 
fix Issue googleapis#11
@hannahrogers-google
Merge branch 'master' of github.com:hannahrogers-google/java-pubsub
7d704f0
@hannahrogers-google
Merge branch 'master' of https://github.com/googleapis/java-pubsub
9e0ebc6
@hannahrogers-google
feat: Adding support for DLQs
01d41b7 
Adding delivery attempt count to PubsubMessages as a message attribute,
and creating helper function to allow users to get the count without
knowing implementation details.
@hannahrogers-google
Fix formatting
ed3b1ee
@hannahrogers-google
fix: making changes requested in pull request
72f7996
@hannahrogers-google
Merge branch 'master' of https://github.com/googleapis/java-pubsub
389cb86
@hannahrogers-google
Merge branch 'master' of https://github.com/googleapis/java-pubsub
c9e4bd2
@hannahrogers-google
fix: creating fix to not populate delivery attempt attribute when dead
3738ec8 
lettering is not enabled
@hannahrogers-google
Merge branch 'master' of https://github.com/googleapis/java-pubsub
4ce1d3b
@hannahrogers-google
Adding unit test for case in which a received message has no delivery…
aecd4ca 
… attempt
@hannahrogers-google
Making MessageWaiter class more generic to also be used for outstanding
77fa3b3 
ack operations
@hannahrogers-google
Merge branch 'master' of https://github.com/googleapis/java-pubsub
b87023c
@hannahrogers-google
Merge branch 'master' into fix-subscriber-stop
7086b3c
@hannahrogers-google
Waiting for acks to complete before shutting down a streaming subscriber
da0e355 
connection
@hannahrogers-google
Fixing formatting error
dc55fc1
@hannahrogers-google
Merge branch 'master' of https://github.com/googleapis/java-pubsub
45beee2
@hannahrogers-google
fix: making sure all publishes complete before shutting down the
548b7a7 
publisher
@hannahrogers-google
Merge branch 'master' of https://github.com/googleapis/java-pubsub
76a1de9
@hannahrogers-google
Merge branch 'master' of https://github.com/googleapis/java-pubsub
a8989f1
@hannahrogers-google
adding default max outstanding request bytes
b4439fd
@hannahrogers-google
Merge branch 'master' of https://github.com/googleapis/java-pubsub
01f607c
@hannahrogers-google
Merge branch 'master' of https://github.com/googleapis/java-pubsub
c0cddb3
@hannahrogers-google
fix: make push endpoint valid https
9a6381d
@hannahrogers-google
Merge branch 'master' of https://github.com/googleapis/java-pubsub
0bd165f
@hannahrogers-google
Merge branch 'master' of https://github.com/googleapis/java-pubsub
5d47be8
@hannahrogers-google
Merge branch 'master' of https://github.com/googleapis/java-pubsub in…
e3b7f6d 
…to master
@hannahrogers-google
Merge branch 'master' of https://github.com/googleapis/java-pubsub in…
6c1a26c 
…to master
@hannahrogers-google
fix: use default zero value if a flow control setting is not provided
f2fa6b8
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Jul 23, 2020
@codecov
Copy link

codecov bot commented Jul 23, 2020 •
edited
Loading

Codecov Report

Merging #292 into master will increase coverage by 0.12%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #292      +/-   ##
============================================
+ Coverage     79.18%   79.31%   +0.12%     
- Complexity      317      319       +2     
============================================
  Files            21       21              
  Lines          2888     2891       +3     
  Branches        155      155              
============================================
+ Hits           2287     2293       +6     
+ Misses          537      535       -2     
+ Partials         64       63       -1
Impacted Files Coverage Δ Complexity Δ
...cloud/pubsub/v1/StreamingSubscriberConnection.java 65.94% <100.00%> (+0.75%) 11.00 <2.00> (+2.00)
.../com/google/cloud/pubsub/v1/MessageDispatcher.java 85.36% <0.00%> (+0.48%) 26.00% <0.00%> (ø%)
...in/java/com/google/cloud/pubsub/v1/Subscriber.java 81.05% <0.00%> (+1.05%) 22.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 c8f981b...0cd08dc. Read the comment docs.

@hannahrogers-google hannahrogers-google merged commit d56155b into googleapis:master Jul 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kamalaboulhosn kamalaboulhosn kamalaboulhosn approved these changes

Assignees

No one assigned

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Unable to start the subscription with partial flowcontrol details

2 participants

@hannahrogers-google @kamalaboulhosn