Fix: Errno::ECONNRESET (MAYBE-MARKETING-2M) #226
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Based on the error
Errno::ECONNRESET: Connection reset by peer
, I'm implementing a more robust Redis connection configuration in the production environment. This error typically occurs when a network connection is unexpectedly terminated, which is common with Redis connections in production environments under load.The changes focus on the Redis cache store configuration to add connection timeout parameters and reconnection logic:
reconnect_attempts: 1
to try reconnecting once if the connection failsconnect_timeout: 20
to allow up to 20 seconds for initial connectionread_timeout: 0.2
andwrite_timeout: 0.2
to prevent hanging operationsThese timeouts are carefully chosen based on common production scenarios:
This configuration helps prevent connection reset issues by:
Tip
You can make revisions or ask questions of Revise.dev by using
/revise
in any comment or review!/revise Add a comment above the method to explain why we're making this change.
/revise Why did you choose to make this change specifically?