| From: | Simon Riggs <simon(at)2ndQuadrant(dot)com> | 
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> | 
| Cc: | pgsql-hackers(at)postgresql(dot)org | 
| Subject: | Re: TOAST code ignores freespace (was Tweak TOAST code) | 
| Date: | 2010-05-02 15:20:55 | 
| Message-ID: | 1272813656.4161.31585.camel@ebony | 
| Views: | Whole Thread | Raw Message | Download mbox | Resend email | 
| Thread: | |
| Lists: | pgsql-committers pgsql-hackers | 
On Sun, 2010-05-02 at 10:34 -0400, Tom Lane wrote:
> Simon Riggs <simon(at)2ndQuadrant(dot)com> writes:
> > Not commenting further on that patch, but I notice that when we UPDATE
> > the toasting algorithm takes no account of the available freespace on
> > the current block. If we are updating and the space available would make
> > a difference to the row length chosen, it seems like it would be more
> > beneficial to trim the row and encourage HOT updates.
> 
> That doesn't strike me as a terribly good idea: it would make the
> behavior of TOAST significantly more difficult to predict.  Also, what
> happens if we force a row to a smaller size and then it doesn't fit
> anyway (eg because someone else inserted another row on the page while
> we were busy doing this)?  Spend even more cycles to un-toast back to
> the normal size, to be consistent with ordinary cross-page updates?
> 
> Pretty much every previous discussion of tweaking the TOAST behavior
> has focused on giving the user more control (indeed, the patch you
> mention could be seen as doing that).  What you're suggesting here
> would give the user less control, as well as less predictability.
As long as we've considered it, I'm happy either way. You know I'm
happier with more user control.
-- 
 Simon Riggs           www.2ndQuadrant.com
| From | Date | Subject | |
|---|---|---|---|
| Next Message | User Fxjr | 2010-05-02 15:31:41 | npgsql - Npgsql2: [#1010796] Notification thread not released after | 
| Previous Message | Tom Lane | 2010-05-02 14:34:43 | Re: TOAST code ignores freespace (was Tweak TOAST code) | 
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2010-05-02 15:39:39 | Re: pg_migrator to /contrib in a later 9.0 beta | 
| Previous Message | Tom Lane | 2010-05-02 14:34:43 | Re: TOAST code ignores freespace (was Tweak TOAST code) |