[Catalyst] Plugin::Cache silent failures

[Tomas Doran]

Apologies for the delay, I'm a little behind on things this week..

Bill Moseley wrote:
> Unfortunately, Cache::Entry doesn't really specify a return value for
> set(), but common backends like Memcached and FastMmap do seem to
> return a value indicating success.  Not sure if there are other
> commonly used backends that would not have a meaningful return value.
> 
> The plugin doesn't define the return from cache_set, either.
> As it is now, cache_set can silently fail.

It's useful to know the return value in some cases, so I'm all for a 
patch which propagates that.

> I wonder if it would be safe to warn on ->set() failing in the plugin.

I think for the common case, this is probably good behavior, however I 
think there should be a config setting which could disable this warning..

Patches for both of the above would be welcome.

Cheers
t0m