From: Gregory Stark <stark@enterprisedb.com>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: pgsql-patches@postgreSQL.org
Cc: Jan Wieck <JanWieck@Yahoo.com>
Subject: Re: Proposed patch to change TOAST compression strategy
Date: Mon, 18 Feb 2008 10:33:38 +0000
Message-ID: <87ir0mo8ot.fsf@oxford.xeocode.com> (raw)
In-Reply-To: <26048.1203271939@sss.pgh.pa.us>
References: <26048.1203271939@sss.pgh.pa.us>
"Tom Lane" <tgl@sss.pgh.pa.us> writes:
> * Adds an early-failure path to the compressor as suggested by Jan:
> if it's been unable to find even one compressible substring in the
> first 1KB (parameterizable), assume we're looking at incompressible
> input and give up. (There are various ways this could be done, but
> this way seems to add the least overhead to the compressor's inner
> loop.)
I'm not sure how to test the rest of it, but this bit seems testable. I fear
this may be too conservative. Even nigh incompressible data will find a few
backreferences.
I'll try some tests and see.
--
Gregory Stark
EnterpriseDB http://www.enterprisedb.com
Ask me about EnterpriseDB's On-Demand Production Tuning
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-patches@postgresql.org
Cc: stark@enterprisedb.com, tgl@sss.pgh.pa.us, JanWieck@Yahoo.com
Subject: Re: Proposed patch to change TOAST compression strategy
In-Reply-To: <87ir0mo8ot.fsf@oxford.xeocode.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox