-
Notifications
You must be signed in to change notification settings - Fork 838
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
Memory leak #79
Comments
Related: #44 |
it's a pretty bad memory leak and caused us to stop using this module for now. |
Curious if there's an update on this? |
See #82. I'm working on using a new precomputed data format which should solve this issue. |
Thanks for the update! Will put comments on that PR. |
This is fixed in the rewrite and will go out in the |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I am still tracking this down, but I am pretty sure the memoize stuff added in 803d2e2 introduced memory leaking. If you need to format a million unique dates, it's basically going to cache all of them and never let go.
The text was updated successfully, but these errors were encountered: