Code review comment for lp:~osomon/oxide/simple-cache-backend

Revision history for this message
Chris Coulson (chrisccoulson) wrote :

I think this mostly looks ok - the only thing I am concerned about is that clearing the appcache might be racey. So we can get this in to 1.7, I wonder whether we should just switch the network cache for now (given that's the problematic one) and then look at the application cache in 1.8. WDYT?

« Back to merge proposal