identity_cache icon indicating copy to clipboard operation
identity_cache copied to clipboard

Remove :idc_cached_nil

Open fbogsany opened this issue 10 years ago • 1 comments

@dylanahsmith : "I think the only thing that prevented us from storing nil was that cache.read would return nil for a cache miss as well as if the value cached was nil. I think we may even be able to get rid of map_cached_nil_for now that we don't use cache.read."

Follow-up from #154

Note that support for cache.read was re-added in #161

fbogsany avatar Jun 11 '14 14:06 fbogsany

If you still want to remove :idc_cached_nil, then you could replace cache.read with cache.read_multi for a single key. Otherwise, we could just close this issue now that we are supporting ActiveSupport::Cache::MemoryStore.

dylanahsmith avatar Jun 11 '14 19:06 dylanahsmith