Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.3k
Added redeliver_timeout and claim_interval options#12976
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.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Changes fromall commits
File filter
Filter by extension
Conversations
Uh oh!
There was an error while loading.Please reload this page.
Jump to
Uh oh!
There was an error while loading.Please reload this page.
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1038,9 +1038,9 @@ a running Redis server (^5.0). | ||
| A number of options can be configured via the DSN or via the ``options`` key | ||
| under the transport in ``messenger.yaml``: | ||
| =================== ====================================================================== | ||
| Option DescriptionDefault | ||
| =================== ====================================================================== | ||
| stream The Redis stream name messages | ||
| group The Redis consumer group name symfony | ||
| consumer Consumer name used in Redis consumer | ||
| @@ -1056,7 +1056,23 @@ stream_max_entries The maximum number of entries which ``0`` (which means "n | ||
| it to a large enough number to | ||
| avoid losing pending messages | ||
| tls Enable TLS support for the connection false | ||
| redeliver_timeout Timeout before retrying a pending ``3600`` | ||
| message which is owned by an | ||
| abandoned consumer (if a worker died | ||
| for some reason, this will occur, | ||
| eventually you should retry the | ||
| message) - in seconds. | ||
| claim_interval Interval on which pending/abandoned ``60000`` (1 Minute) | ||
| messages should be checked for to | ||
| claim - in milliseconds | ||
| =================== ===================================== ================================= | ||
| .. caution:: | ||
| There should never be more than one `messenger:consume` command running with the same | ||
| config (stream, group and consumer name) to avoid having a message handled more than once. | ||
| Using the ``HOSTNAME`` as the consumer might often be a good idea. In case you are using | ||
| Kubernetes to orchestrate your containers, consider using a ``StatefulSet``. | ||
Contributor There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more. Contributor There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more. Well, with this PR it's not really relevant anymore. You can use a regular Contributor There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more. Yeah thats correct still think its good to have the hostname which doesn't change as it avoids spaming redis with new consumers as they recommend to reuse consumer names. Contributor There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more. Then that should probably be mentioned here but other than that, there's nothing to add regarding k8s :) | ||
| .. tip:: | ||