Re: Teach pg_receivewal to use lz4 compression

From: gkokolatos(at)pm(dot)me
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: Magnus Hagander <magnus(at)hagander(dot)net>, Dilip Kumar <dilipbalaut(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Teach pg_receivewal to use lz4 compression
Date: 2021-11-02 11:36:49
Message-ID: 5r8vOVRcARUs7b2UytrMzvnm5zf3x2hAEokZWHIpz8dcxJVk_FOilVlv1rHiEoBtqrGtqx6ebeAZKwRFT8IxNdelI38aF04H0x2QFQvuzoE=@pm.me
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐

On Tuesday, November 2nd, 2021 at 9:51 AM, Michael Paquier <michael(at)paquier(dot)xyz> wrote:

> On Tue, Nov 02, 2021 at 07:27:50AM +0900, Michael Paquier wrote:
> > On Mon, Nov 01, 2021 at 08:39:59AM +0000, gkokolatos(at)pm(dot)me wrote:
> > > Agreed.
> > >
> > > I have already started on v8 of the patch with that technique. I should
> > > be able to update the thread soon.
> >
> > Nice, thanks!
>

A pleasure. Please find it in the attached v8-0002 patch.

> By the way, I was reading the last version of the patch today, and
> it seems to me that we could make the commit history if we split the
> patch into two parts:
> - One that introduces the new option --compression-method and
> is_xlogfilename(), while reworking --compress (including documentation
> changes).
> - One to have LZ4 support.

Agreed.

>
> v7 has been using "gzip" for the option name, but I think that it
> would be more consistent to use "zlib" instead.

Done.

Cheers,
//Georgios

> --
> Michael

Attachment Content-Type Size
v8-0001-Teach-pg_receivewal-to-use-LZ4-compression.patch text/x-patch 21.8 KB
v8-0002-Teach-pg_receivewal-to-use-LZ4-compression.patch text/x-patch 19.2 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Ajin Cherian 2021-11-02 11:44:05 Re: row filtering for logical replication
Previous Message Ekaterina Sokolova 2021-11-02 11:32:59 Re: RFC: Logging plan of the running query