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 a TTL to transaction cache entries #41

Closed
edaniszewski opened this issue Feb 22, 2018 · 0 comments
Closed

add a TTL to transaction cache entries #41

edaniszewski opened this issue Feb 22, 2018 · 0 comments
Assignees

Comments

@edaniszewski
Copy link
Contributor

Description

Reading through the code, it looks like there is no TTL set when transactions are added to the transaction_cache. We'll probably want that to prevent it from getting massive..

My initial thought is that the TTL can be configurable, and if it doesn't exist in the config, it would fall back to some default. Just need to figure out what a sane default value is.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant