Re: Missed bet in toaster routines - Mailing list pgsql-hackers

From Tom Lane
Subject Re: Missed bet in toaster routines
Date
Msg-id 5576.1074278640@sss.pgh.pa.us
Whole thread Raw
In response to Re: Missed bet in toaster routines  (Jan Wieck <JanWieck@Yahoo.com>)
List pgsql-hackers
Jan Wieck <JanWieck@Yahoo.com> writes:
> Tom Lane wrote:
>> Wouldn't it be a win for heap_tuple_toast_attrs() to fall out quickly
>> if the tuple contains no varlena attributes?

> Why sure it would. That code is quite frequently called.

I take it back --- the short-circuit test already exists, but it's at
the call sites rather than in tuptoaster.c.

However, it looks like we can tighten the tests a bit.  For instance
heap_delete invokes the toaster if HeapTupleHasExtended, whereas it
seems to me it'd be sufficient to check HasExternal --- we don't care
about deleting compressed-in-line stuff, do we?
        regards, tom lane


pgsql-hackers by date:

Previous
From: "scott.marlowe"
Date:
Subject: Re: cache control?
Next
From: Mark Kirkwood
Date:
Subject: Re: FYI , Intel CC and PostgreSQL , benchmark by pgsql