Hi all,

Reading the source for the Stats plugin the other day I noticed a
fairly severe memory leak - the attached one line patch fixes this and
seems to have resolved the OOM issues we were having with perlbal.

Thanks,
Jonty Wareing

--
Developer
Last.fm

[ Post a follow-up to this message ]

Beautiful.  I was just about to make our perlbal run with
Devel::GC::Helper to find this.  :-)

--
http://develooper.com

On Jul 26, 2007, at 11:11 AM, Jonty <jonty.wareing@gmail.com> wrote:

> Hi all,
>
> Reading the source for the Stats plugin the other day I noticed a
> fairly severe memory leak - the attached one line patch fixes this and
> seems to have resolved the OOM issues we were having with perlbal.
>
> Thanks,
> Jonty Wareing
>
> --
> Developer
> Last.fm
> <perlbal_stats_memleak_patch.diff>

[ Post a follow-up to this message ]

Jonty,

Thanks!  Mark checked this in for you... will be in the next release.

- Brad


On Thu, 26 Jul 2007, Jonty wrote:

> Hi all,
>
> Reading the source for the Stats plugin the other day I noticed a
> fairly severe memory leak - the attached one line patch fixes this and
> seems to have resolved the OOM issues we were having with perlbal.
>
> Thanks,
> Jonty Wareing
>
> --
> Developer
> Last.fm
>

[ Post a follow-up to this message ]