Re: Teach pg_receivewal to use lz4 compression
gkokolatos@pm.me
From: gkokolatos@pm.me
To: Michael Paquier <michael@paquier.xyz>
Cc: Magnus Hagander <magnus@hagander.net>,
Dilip Kumar <dilipbalaut@gmail.com>,
PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2021-09-17T08:12:42Z
Lists: pgsql-hackers
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐ On Friday, September 17th, 2021 at 09:39, Michael Paquier <michael@paquier.xyz> wrote: > On Thu, Sep 16, 2021 at 03:17:15PM +0000, gkokolatos@pm.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
Commits
-
Fix header inclusion order in pg_receivewal.c
- f512efb2d50a 15.0 landed
-
Remove useless LZ4 system call on failure when writing file header
- f79962d8264b 15.0 landed
-
Add support for LZ4 compression in pg_receivewal
- babbbb595d23 15.0 landed
-
Rework compression options of pg_receivewal
- d62bcc8b07f9 15.0 landed
-
Clarify some errors in pg_receivewal when closing WAL segments
- cddcf7842c31 15.0 landed