Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cache session active_at in Redis for periodic job processor #340

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jace
Copy link
Member

@jace jace commented Jun 3, 2016

This branch introduces two new patterns that need some discussion and consensus on:

  1. Models that store frequently changing data (like active_at timestamps) in Redis, for periodic job processors to flush to database. Should we override the column property to transparently write to Redis instead of custom code each time? Should the flush to db have a standardised name like say flush_from_redis?
  2. Models that use UUID keys and must expect queries with either UUID or string parameters. Where exactly is type safety enforced?

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.

1 participant