On 2014-07-04 15:34:14 +0530, Abhijit Menon-Sen wrote: > At 2014-07-04 11:34:21 +0200, and...@2ndquadrant.com wrote: > > > > So we're leaking memory here? For --stats that could well be > > relevant... > > Will fix (I think using a static buffer would be OK here).
In that place, yes. There there's several other places where that's probably isn't going to work. Probably makes sense to check memory usage after running it over a larger amount of WAL. > > It's far from guaranteed that 27 will always suffice. I guess it's ok > > because it doesn't cause bad breakage, but just some misalignment... > > Yes, that was my thought too. > > I could measure the lengths of things and align columns dynamically, but > I really doubt it's worth the effort in this case. Nah. Too much work for no gain ;) Greetings, Andres Freund -- Andres Freund http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (firstname.lastname@example.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers