[Resource Leak] Redis connection pool never disconnected on shutdown#2887
[Resource Leak] Redis connection pool never disconnected on shutdown#2887sanrishi wants to merge 1 commit into
Conversation
- Fix CacheService.close() to disconnect the connection pool (pool.disconnect()) - Add cleanup_cache_service() module-level function - Register app shutdown handler in main.py to close Redis on exit - Prevents lingering Redis connections on process restart Fixes ritesh-1918#2874
|
Someone is attempting to deploy a commit to the ritesh Team on Vercel. A member of the Team first needs to authorize it. |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
1 participant
Adds pool.disconnect() and app shutdown hook to gracefully close Redis connections. Fixes #2874