Re: Compressed TOAST Slicing
Tom Lane <tgl@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Paul Ramsey <pramsey@cleverelephant.ca>
Cc: Stephen Frost <sfrost@snowman.net>, pgsql-hackers@lists.postgresql.org
Date: 2018-11-01T23:02:26Z
Lists: pgsql-hackers
Commits
Same data as JSON:
GET /api/v1/messages/:b64id/commits
the thread's linked commits as JSON, with link sources.
API reference →
-
Add support for partial TOAST decompression
- 4d0e994eed83 12.0 landed
-
Remove remaining hard-wired OID references in the initial catalog data.
- 3aa0395d4ed3 12.0 cited
-
Rephrase references to "time qualification".
- ebcc7bf949ba 12.0 cited
Paul Ramsey <pramsey@cleverelephant.ca> writes: > On Thu, Nov 1, 2018 at 2:29 PM Stephen Frost <sfrost@snowman.net> wrote: >> and secondly, why we wouldn't consider >> handling a non-zero offset. A non-zero offset would, of course, still >> require decompressing from the start and then just throwing away what we >> skip over, but we're going to be doing that anyway, aren't we? Why not >> stop when we get to the end, at least, and save ourselves the trouble of >> decompressing the rest and then throwing it away. > I was worried about changing the pg_lz code too much because it scared me, > but debugging some stuff made me read it more closely so I fear it less > now, and doing interior slices seems not unreasonable, so I will give it a > try. I think Stephen was just envisioning decompressing from offset 0 up to the end of what's needed, and then discarding any data before the start of what's needed; at least, that's what'd occurred to me. It sounds like you were thinking about hacking pg_lz to not write the leading data anywhere. While that'd likely be a win for cases where there was leading data to discard, I'm worried about adding any cycles to the inner loops of the decompressor. We don't want to pessimize every other use of pg_lz to buy a little bit for these cases. regards, tom lane