Skip to content

Comments

[1.17] Docs redis honor resiliency#5044

Open
cicoyle wants to merge 1 commit intodapr:v1.17from
cicoyle:feat-redis-honor-resiliency
Open

[1.17] Docs redis honor resiliency#5044
cicoyle wants to merge 1 commit intodapr:v1.17from
cicoyle:feat-redis-honor-resiliency

Conversation

@cicoyle
Copy link
Contributor

@cicoyle cicoyle commented Feb 19, 2026

docs for this PR noting how we honor resiliency policy now and not redis infinite redelivery

…olicy

Signed-off-by: Cassandra Coyle <cassie@diagrid.io>
@cicoyle cicoyle requested review from a team as code owners February 19, 2026 18:01
Copy link
Member

@msfussell msfussell left a comment

Choose a reason for hiding this comment

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

@cicoyle - Left comments

## Resiliency and redelivery

The Redis Streams pubsub component honors the [Dapr resiliency policy]({{% ref "resiliency-overview" %}}). After your
application handler returns (success, error, RETRY, or DROP), the component acknowledges the message to Redis. Redis
Copy link
Member

Choose a reason for hiding this comment

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

Any reason some are CAPITALs and others are not?

The Redis Streams pubsub component honors the [Dapr resiliency policy]({{% ref "resiliency-overview" %}}). After your
application handler returns (success, error, RETRY, or DROP), the component acknowledges the message to Redis. Redis
then stops redelivering that message. Retries and dead-letter behavior are controlled by your resiliency policy (for
example, `maxRetries` and retry duration), not by Redis indefinitely.
Copy link
Member

Choose a reason for hiding this comment

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

what does "not by Redis indefinitely" mean? Not sure what this is implying. Can you clarify

@marcduiker marcduiker added this to the 1.17 milestone Feb 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants