Skip to content
This repository has been archived by the owner on Jan 16, 2023. It is now read-only.

the default MAX_ENTRIES cache setting (300) is probably worth mentioning in the README #47

Closed
benrobster opened this issue Jul 30, 2014 · 2 comments

Comments

@benrobster
Copy link

I realize its not a collectfast problem, but I spent a long time trying to figure out why caching was only working for a (seemingly random) portion of my static files. Eventually I found the MAX_ENTRIES default setting for Django caches, which is set to 300 by default (I think this only applies to DB or File Based caching, which is what i'm using for my CollectFast cache).

It may be useful for some users to call out the bad default on the "MAX_ENTRIES" option when you call out "TIMEOUT" in the README, since they probably won't be familiar with it and it will almost always reduce the effectiveness of CollectFast if used with affected cache types, since only 300 file hashes will be cached

@antonagestam
Copy link
Owner

@nebstrebor Please feel free to create a pull request for this. I definitely think this makes sense as this package is targeted to projects with large amounts of static files.

@browniebroke
Copy link
Contributor

Created a pull request as I have the issue, and I would have missed it if not checking the issues section.

antonagestam added a commit that referenced this issue May 19, 2015
Mention MAX_ENTRIES setting in README closes #47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants