Re: patch submission: truncate trailing nulls from heap rows to reduce the size of the null bitmap [Review] - Mailing list pgsql-hackers

From Greg Stark
Subject Re: patch submission: truncate trailing nulls from heap rows to reduce the size of the null bitmap [Review]
Date
Msg-id CAM-w4HOiT8actZr_HUeoB7m+ddU91_fZaeLEE-7PKijKTPa+Ew@mail.gmail.com
Whole thread Raw
In response to Re: Re: patch submission: truncate trailing nulls from heap rows to reduce the size of the null bitmap [Review]  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: patch submission: truncate trailing nulls from heap rows to reduce the size of the null bitmap [Review]
List pgsql-hackers
On Thu, Jan 24, 2013 at 8:17 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> As I said, I'd be willing to take this risk if the patch showed more
> attractive performance benefits ... but it still seems mighty marginal
> from here.

I think the benchmarks given so far are actually barking up the wrong
tree though. There are usage patterns that some people do engage in
that involve dozens or hundreds of columns that are mostly NULL. If
they're saving 10-20 bytes per row that's not insignificant. And they
could be saving much more than that.

That said I don't know just how common that usage pattern is. And I'm
not sure how many of those people would be able to arrange for the
null columns to land at the end of the row.

It's a bit frustrating because it does seem like if it had been
written this way to begin with nobody would every have questioned
whether it was a good idea and nobody would ever have suggested
ripping it out.


-- 
greg



pgsql-hackers by date:

Previous
From: Andres Freund
Date:
Subject: Re: logical changeset generation v4 - Heikki's thoughts about the patch state
Next
From: Andres Freund
Date:
Subject: Re: logical changeset generation v4 - Heikki's thoughts about the patch state