Bug #80697 for Geo-Coder-Many: wrong cache usage

Wed Nov 07 18:37:23 2012 ZUMMO [...] cpan.org - Ticket created

Subject:

wrong cache usage

Storing complex structures,like hashes, should be done with ->freeze/thaw instead of ->set/get.

Sat Nov 10 05:06:54 2012 kaoru [...] slackwise.net - Correspondence added

On Wed Nov 07 18:37:23 2012, ZUMMO wrote: Show quoted text

> Storing complex structures,like hashes, should be done > with ->freeze/thaw instead of ->set/get. >

Hi Alessandro, What module are you using for caching? The documentation of Geo::Coder::Many clearly states that whatever caching module you are using should provide get(), set() and remove() methods. It does not currently state that those methods should deal correctly with references - I will update the documentation accordingly shortly. I recommend you use the CHI module (and I will update the documentation to mention that too) as that provides get() and set() which will correctly deal with serialization of reference-based data structures. If you can provide me any other information that would be very helpful. Thanks, - Alex

Sat Nov 10 05:06:55 2012 The RT System itself - Status changed from 'new' to 'open'

Sat Nov 10 06:50:25 2012 ZUMMO [...] cpan.org - Correspondence added

Show quoted text

> What module are you using for caching?

Hi, I used Cache::File which implements Cache and Cache::Entry. Show quoted text

> methods. It does not currently state that those methods should deal > correctly with references - I will update the documentation

accordingly Show quoted text

> shortly.

I guess that would be fine. thanks.

Sat Nov 10 16:31:05 2012 kaoru [...] slackwise.net - Correspondence added

Updated POD and also updated the check that occurs when you create an object instance with a cache object so that it croaks if the cache doesn't deal well with references. https://metacpan.org/release/Geo-Coder-Many Should be fixed in version 0.4

Sat Nov 10 16:31:07 2012 kaoru [...] slackwise.net - Status changed from 'open' to 'resolved'