Skip to content
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

pubsub: support synchronous receive #1088

Closed
jba opened this issue Aug 7, 2018 · 4 comments
Closed

pubsub: support synchronous receive #1088

jba opened this issue Aug 7, 2018 · 4 comments
Assignees
@jba
Labels
api: pubsub Issues related to the Pub/Sub API. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. 🚨 This issue needs some love. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.

Comments

@jba
Copy link
Contributor

jba commented Aug 7, 2018

Provide an option to Subscriber.Receive that pulls messages synchronously, rather than in the background via streaming.

This would result in lower throughput, but would also guarantee that the number of pulled messages would not exceed MaxOutstandingMessages.

Low-throughput applications might prefer this mode.
@jba jba added type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. api: pubsub Issues related to the Pub/Sub API. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. labels Aug 7, 2018
@jba jba self-assigned this Aug 7, 2018
@jba jba mentioned this issue Aug 10, 2018
Closed
@JustinBeckwith JustinBeckwith added 🚨 This issue needs some love. and removed 🚨 This issue needs some love. labels Aug 12, 2018
@JustinBeckwith JustinBeckwith added 🚨 This issue needs some love. and removed 🚨 This issue needs some love. labels Aug 22, 2018
@JustinBeckwith JustinBeckwith added 🚨 This issue needs some love. and removed 🚨 This issue needs some love. labels Sep 1, 2018
@dthorsen
Copy link

dthorsen commented Sep 7, 2018

@jba We are also hitting this issue when we have a large backlog of small messages. Our symptoms are nearly identical to the issue in #1097. Has work begun on this feature? Any idea when you think it might be released?

@jba
Copy link
Contributor Author

jba commented Sep 9, 2018

Aiming for end of September.

@JustinBeckwith JustinBeckwith added 🚨 This issue needs some love. and removed 🚨 This issue needs some love. labels Sep 14, 2018
@jba jba mentioned this issue Sep 28, 2018
Closed
@cmoad
Copy link

cmoad commented Oct 5, 2018

Just deployed this. We will report back here soon. Thanks!

@cmoad
Copy link

cmoad commented Oct 5, 2018 •
edited
Loading

Things appear to be running smooth. See attached screenshot. You can see when we deployed by the switch in pull type.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jba jba

Labels
api: pubsub Issues related to the Pub/Sub API. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. 🚨 This issue needs some love. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@JustinBeckwith @cmoad @dthorsen @jba