Skip to content

WIP - Shared redis subscribe#4432

Closed
benedikt-bartscher wants to merge 5 commits intoreflex-dev:mainfrom
benedikt-bartscher:shared-redis-subscribe
Closed

WIP - Shared redis subscribe#4432
benedikt-bartscher wants to merge 5 commits intoreflex-dev:mainfrom
benedikt-bartscher:shared-redis-subscribe

Conversation

@benedikt-bartscher
Copy link
Copy Markdown
Contributor

@benedikt-bartscher benedikt-bartscher commented Nov 25, 2024

based on #4419
not sure if we really want this

@masenf
Copy link
Copy Markdown
Collaborator

masenf commented Oct 27, 2025

The spirit of this is also fixed by #5932, which optimizes the redis lock path:

  • 1 pubsub per app instance
  • create asyncio.Event to signal waiters to redis/lock lock expiration
  • cached in-memory states when instance is holding redis lock with deferred write

@masenf masenf closed this Oct 27, 2025
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.

2 participants