pg.ddx.io  pgsql-hackers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Michael Paquier <michael@paquier.xyz>
To: Tomas Vondra <tomas.vondra@2ndquadrant.com>
Cc: Andres Freund <andres@anarazel.de>
Cc: Andrey Borodin <x4mmm@yandex-team.ru>
Cc: Konstantin Knizhnik <k.knizhnik@postgrespro.ru>
Cc: pgsql-hackers <pgsql-hackers@postgresql.org>
Cc: Vladimir Leskov <vladimirlesk@yandex-team.ru>
Subject: Re: pglz performance
Date: Mon, 5 Aug 2019 16:04:46 +0900
Message-ID: <20190805070446.GG1740@paquier.xyz> (raw)
In-Reply-To: <20190802175239.5ds6rhdso6hwy4vk@development>
References: <469C9ED9-348C-4FE7-A7A7-B0FA671BEE4C@yandex-team.ru>
	<20190513071427.GB2273@paquier.xyz>
	<169163A8-C96F-4DBE-A062-7D1CECBE9E5D@yandex-team.ru>
	<ea57b49a-ecf0-481a-a77b-631833354f7d@postgrespro.ru>
	<20190802144345.d62jtiyyx6r2y73f@development>
	<DBB2A9E5-29FD-40BF-AC60-BD990FBF142F@yandex-team.ru>
	<20190802163948.i6mjypdgujeorrbi@alap3.anarazel.de>
	<20190802170039.o4pabnzm4xy3z7uj@development>
	<20190802171258.nob5zo2vu7siqo2b@alap3.anarazel.de>
	<20190802175239.5ds6rhdso6hwy4vk@development>

On Fri, Aug 02, 2019 at 07:52:39PM +0200, Tomas Vondra wrote:
> On Fri, Aug 02, 2019 at 10:12:58AM -0700, Andres Freund wrote:
>> Why would they be stuck continuing to *compress* with pglz? As we
>> fully retoast on write anyway we can just gradually switch over to the
>> better algorithm. Decompression speed is another story, of course.
> 
> Hmmm, I don't remember the details of those patches so I didn't realize
> it allows incremental recompression. If that's possible, that would mean
> existing systems can start using it. Which is good.

It may become a problem on some platforms though (Windows?), so
patches to improve either the compression or decompression of pglz are
not that much crazy as they are still likely going to be used, and for
read-mostly switching to a new algo may not be worth the extra cost so
it is not like we are going to drop it completely either.  My take,
still the same as upthread, is that it mostly depends on the amount of
complexity each patch introduces compared to the performance gain. 

> Another question is whether we'd actually want to include the code in
> core directly, or use system libraries (and if some packagers might
> decide to disable that, for whatever reason).

Linking to system libraries would make our maintenance much easier,
and when it comes to have a copy of something else in the tree we
would be stuck with more maintenance around it.  These tend to rot
easily.  After that comes the case where the compression algo is not
in the binary across one server to another, in which case we have an
automatic ERROR in case of a mismatching algo, or FATAL for
deompression of FPWs at recovery when wal_compression is used.
--
Michael

Attachments:

  signature.asc (application/pgp-signature, 832B) download

view thread (70+ messages)  latest in thread

Message-ID: <20190805070446.GG1740@paquier.xyz>
Permalink:  ../20190805070446.GG1740@paquier.xyz/
Also on:    postgresql.org/message-id/20190805070446.GG1740@paquier.xyz

 · 

reply

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-hackers@postgresql.org
  Cc: michael@paquier.xyz, tomas.vondra@2ndquadrant.com, andres@anarazel.de, x4mmm@yandex-team.ru, k.knizhnik@postgrespro.ru, vladimirlesk@yandex-team.ru
  Subject: Re: pglz performance
  In-Reply-To: <20190805070446.GG1740@paquier.xyz>

* 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