You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It should have an if(that.cache) in there to check if the cache is enabled, otherwise the data stored in the cache will never be retrieved and it's just wasting memory. If someone had a lot of static files they would not be happy about all that wasted memory!
Of course as someone pointed out in another ticket, there's a case to be made for eliminating the in-memory cache altogether as reading static files as the OS generally does this caching for you. Caches are more useful for dynamically generated data. Eliminating the in-memory cache would also fix this issue.
The text was updated successfully, but these errors were encountered:
There's a piece of code:
It should have an if(that.cache) in there to check if the cache is enabled, otherwise the data stored in the cache will never be retrieved and it's just wasting memory. If someone had a lot of static files they would not be happy about all that wasted memory!
Of course as someone pointed out in another ticket, there's a case to be made for eliminating the in-memory cache altogether as reading static files as the OS generally does this caching for you. Caches are more useful for dynamically generated data. Eliminating the in-memory cache would also fix this issue.
The text was updated successfully, but these errors were encountered: