search

bbc / alephant-broker

Brokers requests for alephant components
MIT License
4 stars 3 forks source link

Bring in Storage gem #59

Closed samfrench closed 8 years ago

samfrench commented 8 years ago

This is the other place the Cache gem gets used. I've changed this to use Storage. This means we can deprecate the Alephant Cache gem.

samfrench commented 8 years ago

@BBCMarkMcDonnell, I replaced the cache with the storage gem in this PR. I also changed a few of the cache names, as the methods retrieved from the cache and then s3, so a cache_fetch wasn't exactly accurate.

Integralist commented 8 years ago

@samfrench other than a minor comment about double checking consuming apps to see if they reference the changed custom exception this is good to go :cake: