| 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) |