1

Closed

Orchard.Caching DefaultCacheStorageProvider should use the newer MemoryCache

description

Closed Oct 17 at 5:32 PM by Piedone

comments

Jetski5822 wrote Oct 13 at 10:58 PM

I didnt know this existed.... but yes we should!

Piedone wrote Oct 14 at 9:43 PM

Piedone wrote Oct 14 at 10:08 PM

Note: for effective memory management there should be a single MemoryCache instance, preferable through using MemoryCache.Default. See: http://stackoverflow.com/a/13425322/220230

Also MemorCache has regions that aren't implemented. Instead we could use this for clearing all cache entries for specific tenants: http://stackoverflow.com/a/22388943/220230

Jetski5822 wrote Oct 17 at 1:04 PM

Would Regions not be the Tenant?

Piedone wrote Oct 17 at 1:49 PM

It would, but despite the interface containing it, there is no existing implementation for regions.

Piedone wrote Oct 17 at 5:30 PM

Fixed in changeset d5f27f560c42820c095a8b75b01a14f36b9bd00b