Re: memory-related bugs - Mailing list pgsql-hackers

From Noah Misch
Subject Re: memory-related bugs
Date
Msg-id 20110907030358.GA16899@tornado.leadboat.com
Whole thread Raw
In response to Re: memory-related bugs  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
On Tue, Sep 06, 2011 at 03:00:42PM -0400, Tom Lane wrote:
> Noah Misch <noah@leadboat.com> writes:
> > On Sat, Mar 12, 2011 at 12:44:29PM -0500, Tom Lane wrote:
> >> I wonder whether we should instead fix this by copying the correct tuple
> >> length.
> 
> > Seems like a step in the wrong direction.  We only use typlen and typbyval
> > beyond the immediate context.
> 
> Well, the whole point of exposing the pg_type tuple is to avoid making
> assumptions about what parts of it the type-specific analyze routine
> will wish to look at.  If anything we ought to move in the direction of
> allowing the non-fixed fields to be accessible too.  I didn't do that,
> since it would imply an ABI change (to add a HeapTuple field to
> VacAttrStats) and would therefore not be back-patchable.  But I did
> change the code to use SearchSysCacheCopy, which fixes this bug and
> is readily extensible if we do decide to add such a field later.

That is a cleaner approach.  Thanks.

-- 
Noah Misch                    http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services


pgsql-hackers by date:

Previous
From: Andy Colson
Date:
Subject: REVIEW Single pass vacuum - take 2
Next
From: Fujii Masao
Date:
Subject: Re: regular logging of checkpoint progress