Created attachment 610808 [details] [diff] [review] patch limiting size of in-memory collections cache This patch adds a hard limit of 1000 items in the in-memory cache of collection names/ids. Additional collections will continue to work, but will be slower because their details need to be looked up in the database on every request.
Comment on attachment 610808 [details] [diff] [review] patch limiting size of in-memory collections cache Code is fine (no tests?), but my suspicion is that 1000 may end up too low and we may need to go to, say, 5000. Something to keep an eye on once we see the actual table size.
:telliott or :rfkelly - did you make a final decision on 1000 vs. 5000 here?
It's at 1000 in the code, I think it should be safe to leave at that for now. Things will still work even if more than 1000 collections are created, they'll just be a bit slower. And it's trivial to bump this number if required.