Re: Fixes for compression options of pg_receivewal and refactoring of backup_compression.{c,h}
Michael Paquier <michael@paquier.xyz>
From: Michael Paquier <michael@paquier.xyz>
To: gkokolatos@pm.me
Cc: Postgres hackers <pgsql-hackers@lists.postgresql.org>, Robert Haas <robertmhaas@gmail.com>
Date: 2022-04-13T21:18:29Z
Lists: pgsql-hackers
On Wed, Apr 13, 2022 at 02:58:28PM +0000, gkokolatos@pm.me wrote: > It's really not hard to add compression level. However we had briefly > discussed it in the original thread [1] and decided against. That is why > I did not write that code. If the community thinks differently now, let > me know if you would like me to offer a patch for it. The issue back then was how to design the option set to handle all that (right? My memories may be short on that), and pg_basebackup takes care of that with its option design. This is roughly what has been done here, except that this was for the contentSize: https://www.postgresql.org/message-id/rYyZ3Fj9qayyY9-egNsV_kkLbL_BSWcOEdi3Mb6M9eQRTkcA2jrqFEHglLUEYnzWR_wttCqn7VI94MZ2p7mwNje51lHTvWYnJ1jHdOceen4=@pm.me Do you think that the extra test coverage is going to be too much of a burden? I was thinking about just adding a level to the main lz4 command, with an extra negative test in the SKIP block with a level out of range -- Michael
Commits
-
Handle compression level in pg_receivewal for LZ4
- 42e44f3b3830 15.0 landed
-
Rework compression options of pg_receivewal
- 042a923ad53d 15.0 landed
-
Remove WalCompressionMethod in favor of pg_compress_algorithm
- 3603f7c6e66a 15.0 landed
-
Rename backup_compression.{c,h} to compression.{c,h}
- a4b57543acfb 15.0 landed