Skip to content

Conversation

@mukund-ananthu
Copy link
Contributor

@mukund-ananthu mukund-ananthu commented Apr 3, 2024

  1. Setting timeout to be infinite when publishing with ordering keys enabled.
  2. This is required to retry infinitely when the client library receives retriable error codes from the backend.
  3. Note that there is another test test_publish_with_ordering_key_with_no_retry that currently does not test the retry behavior or settings because the kwargs for commit_retry would be None and commit_timeout would be _MethodDefault._DEFAULT_VALUE when no retries are specified. This test will be improved in a subsequent PR.

Fixes #1084 🦕

@mukund-ananthu mukund-ananthu requested review from a team as code owners April 3, 2024 18:12
@product-auto-label product-auto-label bot added size: xs Pull request size is extra small. api: pubsub Issues related to the googleapis/python-pubsub API. labels Apr 3, 2024
@mukund-ananthu mukund-ananthu force-pushed the infiniteRetryOrderingEnabled branch from f8c5928 to 302eb45 Compare April 3, 2024 18:20
@mukund-ananthu mukund-ananthu self-assigned this Apr 3, 2024
@mukund-ananthu mukund-ananthu changed the title Set timeout to infinite for publishing with ordering keys enabled fix: Set timeout to infinite for publishing with ordering keys enabled Apr 3, 2024
@mukund-ananthu mukund-ananthu force-pushed the infiniteRetryOrderingEnabled branch from bc3ce2d to a926ad1 Compare April 3, 2024 19:19
@conventional-commit-lint-gcf
Copy link

conventional-commit-lint-gcf bot commented Apr 3, 2024

🤖 I detect that the PR title and the commit message differ and there's only one commit. To use the PR title for the commit history, you can use Github's automerge feature with squashing, or use automerge label. Good luck human!

-- conventional-commit-lint bot
https://conventionalcommits.org/

@mukund-ananthu mukund-ananthu force-pushed the infiniteRetryOrderingEnabled branch from a926ad1 to 80542ef Compare April 3, 2024 19:21
@mukund-ananthu mukund-ananthu added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 3, 2024
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 3, 2024
@mukund-ananthu mukund-ananthu force-pushed the infiniteRetryOrderingEnabled branch from 80542ef to 2dfdce7 Compare April 3, 2024 19:32
@mukund-ananthu mukund-ananthu force-pushed the infiniteRetryOrderingEnabled branch from 2dfdce7 to 43b0412 Compare April 4, 2024 00:56
@mukund-ananthu mukund-ananthu enabled auto-merge (squash) April 4, 2024 01:02
@mukund-ananthu mukund-ananthu added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 4, 2024
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 4, 2024
@mukund-ananthu mukund-ananthu added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 4, 2024
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 4, 2024
@mukund-ananthu mukund-ananthu merged commit 67daf3c into main Apr 4, 2024
@mukund-ananthu mukund-ananthu deleted the infiniteRetryOrderingEnabled branch April 4, 2024 03:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: pubsub Issues related to the googleapis/python-pubsub API. size: xs Pull request size is extra small.

3 participants