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-09-17 08:12:42 |
Message-ID: | 2sv1K4rAd0hZl65M4_N_yavaqHXnYQJRZOJFK8zMYiHRZsR1BJvXzprxTqi3IzangO5hyuZC5ES-VulHvIU-8B7zJGr15wgF4Dlfbfp2QRk=@pm.me |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
On Friday, September 17th, 2021 at 09:39, Michael Paquier <michael(at)paquier(dot)xyz> wrote:
> On Thu, Sep 16, 2021 at 03:17:15PM +0000, gkokolatos(at)pm(dot)me wrote:
>
> > Hopefully fixed.
>
> Thanks for the new version. I have put my hands on the patch, and
> began reviewing its internals with LZ4. I am not done with it yet,
> and I have noticed some places that could be improved (error handling,
> some uses of LZ4F_flush() that should be replaced LZ4F_compressEnd(),
> and more tweaks). I'll send an updated version once I complete my
> review, but that looks rather solid overall.
Thanks! Looking forward to seeing it!
> The changes done in close_walfile()@receivelog.c are useful taken
> independently, so I have applied these separately.
Yeah, I was considering it to split them over to a separate commit,
then decided against it. Will do so in the future.
Cheers,
//Georgios
> --------------------------------------------------------------------
> Michael
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2021-09-17 09:11:35 | Re: Correct handling of blank/commented lines in PSQL interactive-mode history |
Previous Message | Ronan Dunklau | 2021-09-17 08:09:00 | Re: Patch to avoid orphaned dependencies |