On Tue, Oct 14, 2014 at 11:57 AM, Henrik Lindberg <
henrik.lindb...@cloudsmith.com> wrote:

> On 2014-14-10 17:36, Ben Ford wrote:
>
>> I have to admit that this email made me feel a little bit dumb. Could
>> you provide a TL;DR summary that at least provides a little context for
>> this? Is this something that people writing types, functions, hiera
>> backends, or report processors need to concern themselves with?
>>
>>
> Sorry about that - this is mostly if you are contributing to Puppet
> itself, or if you find that your implementation in Ruby (of whatever) leaks
> memory and you need to find the cause.
>
> If you are following the well beaten path when writing types and providers
> etc. you need not worry. If you are writing caching in any form you are
> potentially causing memory leaks and the tips here apply.
>

Is my understanding that the way caching was implemented, by using a full
resource type object as the Hash key in a class variable, is what caused
the leak?  Specifically, the resource type would never be cleaned up by the
garbage collector because it's a hash key in a class instance variable,
correct?

Here:
https://github.com/puppetlabs/puppet/pull/3188/files#diff-d8c96ce74a7f29debda40750462bd248L249

-Jeff

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to puppet-dev+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/puppet-dev/CAOXx1vHk0o5Hfezg92NwpOtjMcp-uFf2sE%3DY88s%2BWkERd0h0AA%40mail.gmail.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to