← Back to team overview

ourdelta-developers team mailing list archive

Re: fix from Neil Katin for userstatv2

 

I noticed elsewhere a mention of Neil Katin's memory leak fix in 5.0.86. Is
the CLIENT USER_STATISTICS patch described below that patch? (If it is, I
will quit  pursuing the fix, since I have already incorporated his fix in
our source tree. Thanks, Neil!!)

Our system administrators report that there might still be a small memory
leak in our mysql servers--it appears in a few of our boxes. If you are
aware of other memory leaks in any of the Google or Percona patches, I would
appreciate hearing about them, so I can compare the changes with our own
code. The leaks *might* be in vanilla mysql itself ...??

Thanks for everything that all of you do! I really appreciate it!

Weldon Whipple

On Fri, Sep 25, 2009 at 5:38 PM, Arjen Lentz <arjen@xxxxxxxxxxxxx> wrote:

> Hi Vadim
>
> Please merge this in (sorry the paths are from the ourdelta tree, but the
> offset within the file is compatible with yours)
>
> === modified file 'mysql/5.0/percona_maintained/userstatv2.patch'
> --- mysql/5.0/percona_maintained/userstatv2.patch       2009-09-25 23:16:39
> +0000
> +++ mysql/5.0/percona_maintained/userstatv2.patch       2009-09-25 23:28:28
> +0000
> @@ -3901,7 +3901,7 @@
>
>  +ST_FIELD_INFO user_stats_fields_info[]=
>  +{
> -+  {"USER", 16, MYSQL_TYPE_STRING, 0, 0, "User"},
> ++  {"USER", USERNAME_LENGTH, MYSQL_TYPE_STRING, 0, 0, "User"},
>
>
>
> By the way, his fix for the CLIENT field was HOSTNAME_LENGTH (60) whereas
> you used LIST_PROCESS_HOST_LEN (64)
> In the mysql.user table the host field is a CHAR(60), so I'm guessing that
> Neil picked the correct macro.
> Harmless in this case, now, but might as well use the proper macro because
> if anything else changes later it could break.
>
>
> --
> Arjen Lentz, Exec.Director @ Open Query (http://openquery.com)
> Exceptional Services for MySQL at a fixed budget.
>
> Follow our blog at http://openquery.com/blog/
> OurDelta: enhanced builds for MySQL @ http://ourdelta.org
>
>
>
>
>
>
>
>
>
> _______________________________________________
> Mailing list: https://launchpad.net/~ourdelta-developers<https://launchpad.net/%7Eourdelta-developers>
> Post to     : ourdelta-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~ourdelta-developers<https://launchpad.net/%7Eourdelta-developers>
> More help   : https://help.launchpad.net/ListHelp
>

Follow ups

References