Conversation
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
Greptile OverviewGreptile SummaryThis PR adds a Key Changes:
Note: The Sentinel configuration doesn't include the same reconnect handler, which may be intentional but creates inconsistency across deployment modes. Confidence Score: 4/5
Important Files Changed
|
Additional Comments (1)
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time! |
Context
This PR implements redis reconnect when there is a redeployment of redis happens in platforms like aws elasticache. This does a reconnection with the DNS instead of keeping the same connection.
Reference: https://github.com/redis/ioredis?tab=readme-ov-file#reconnect-on-error
Screenshots
Steps to verify the change
Type
Checklist
type(scope): short description(scope is optional, e.g.,fix: prevent crash on syncorfix(api): handle null response).