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

Proposal: Use LRUCache instead of Map #23

Open
dyst5422 opened this issue Mar 9, 2021 · 0 comments
Open

Proposal: Use LRUCache instead of Map #23

dyst5422 opened this issue Mar 9, 2021 · 0 comments

Comments

@dyst5422
Copy link

dyst5422 commented Mar 9, 2021

Using LRU cache https://www.npmjs.com/package/lru-cache would handle the duration and allow for more complex cache eviction policies.

Seems like a no brainer to me UNLESS you are trying to keep zero dependencies - which is a valid desire. In that case, I would recommend having dependency injection of a cache which would allow for this without complication.

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

No branches or pull requests

1 participant