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

add cache Tag #23

Open
alireza5014 opened this issue Jan 5, 2021 · 1 comment
Open

add cache Tag #23

alireza5014 opened this issue Jan 5, 2021 · 1 comment
Labels
enhancement New feature or request

Comments

@alireza5014
Copy link

there is a problem when we try to recache , it has not any tag so if i update my model it changes for everyone so other users need cache again you can add a specific id for cache Model . what do you say ?
thank you.
for example we user 1 and 2 .

user 1 updates his profile because it is not using specific tag for cache Model . it is doing the same for the user 2.

@frankthelen
Copy link
Collaborator

frankthelen commented Jan 9, 2021

Well, sequelize-simple-cache is explicitly a simple cache. It's good for models/tables that are heavily read but rarely written such as system configuration data and alike. So, a write operation will invalidate the entire cache regarding that specific model/table. The current implementation doesn't allow to make this more flexible or more fine grained than on model/table-level. It is, well, just a simple cache for a very specific scenario. If you have specific ideas to improve, please let me know.

@frankthelen frankthelen added the enhancement New feature or request label Feb 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants