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

pass retry configuration to each next_page in Pager calls #888

Open
daniel-sanche opened this issue Mar 12, 2024 · 0 comments
Open

pass retry configuration to each next_page in Pager calls #888

daniel-sanche opened this issue Mar 12, 2024 · 0 comments
Assignees
@daniel-sanche
Labels
api: firestore Issues related to the googleapis/python-firestore API. priority: p3 Desirable enhancement or fix. May not be included in next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Milestone
Retry Audit

Comments

@daniel-sanche
Copy link
Contributor

We have three rpcs that use pagers: ListDocuments, ListCollectionIds, and PartitionQuery

Currently, retry and timeout values can be passed in to all 3, but it is only used in the initial call. When the next_page call is made, it uses API default values for these fields

This should be addressed at the gapic-generator level. See googleapis/gapic-generator-python#1974
@daniel-sanche daniel-sanche added type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. priority: p3 Desirable enhancement or fix. May not be included in next release. labels Mar 12, 2024
@daniel-sanche daniel-sanche added this to the Retry Audit milestone Mar 12, 2024
@daniel-sanche daniel-sanche self-assigned this Mar 12, 2024
@product-auto-label product-auto-label bot added the api: firestore Issues related to the googleapis/python-firestore API. label Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@daniel-sanche daniel-sanche

Labels
api: firestore Issues related to the googleapis/python-firestore API. priority: p3 Desirable enhancement or fix. May not be included in next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Projects
None yet
Milestone
Retry Audit
Development

No branches or pull requests
1 participant
@daniel-sanche