Skip to content
This repository was archived by the owner on Mar 9, 2026. It is now read-only.

feat: return singleton success future for exactly-once methods in Message - #608

Merged
pradn merged 3 commits into
googleapis:mainfrom
pradn:exactly_once_return_singleton
Mar 9, 2022
Merged

feat: return singleton success future for exactly-once methods in Message#608
pradn merged 3 commits into
googleapis:mainfrom
pradn:exactly_once_return_singleton

Conversation

@pradn

@pradnpradn commented Mar 8, 2022

Copy link
Copy Markdown
Contributor

This is a small change to return a singleton success future that's constructed once instead of making a new future every time for *_with_response methods. We don't want to guarantee that we'll always return a distinct future object. This issue was raised in googleapis/python-pubsublite#312.

@pradn
pradn requested review from a teamMarch 8, 2022 21:00
@product-auto-labelproduct-auto-labelBot added the api: pubsub Issues related to the googleapis/python-pubsub API. label Mar 8, 2022
@pradnpradn changed the title Return singleton success future for exactly-once methods in Messagefeat: return singleton success future for exactly-once methods in MessageMar 8, 2022

@mmicatkammicatka left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@acocuzzoacocuzzo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@pradn
pradn merged commit 253ced2 into googleapis:mainMar 9, 2022
This was referenced Mar 9, 2022
@release-pleaserelease-pleaseBot mentioned this pull request Aug 11, 2022
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: pubsubIssues related to the googleapis/python-pubsub API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@pradn@mmicatka@dpcollins-google@acocuzzo