| From: | Jeevan Ladhe <jeevan(dot)ladhe(at)enterprisedb(dot)com> |
|---|---|
| To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
| Cc: | Dipesh Pandit <dipesh(dot)pandit(at)gmail(dot)com>, Andrew Dunstan <andrew(at)dunslane(dot)net>, tushar <tushar(dot)ahuja(at)enterprisedb(dot)com>, Michael Paquier <michael(at)paquier(dot)xyz>, Dmitry Dolgov <9erthalion6(at)gmail(dot)com>, Mark Dilger <mark(dot)dilger(at)enterprisedb(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: refactoring basebackup.c |
| Date: | 2022-01-28 17:48:15 |
| Message-ID: | CAOgcT0PJpOiafsmZfGZRLGK1WUqZwYdjFWRwgZTVDQHCCwO-EQ@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi Robert,
I have attached the latest rebased version of the LZ4 server-side
compression
patch on the recent commits. This patch also introduces the compression
level
and adds a tap test.
Also, while adding the lz4 case in the pg_verifybackup/t/008_untar.pl, I
found
an unused variable {have_zlib}. I have attached a cleanup patch for that as
well.
Please review and let me know your thoughts.
Regards,
Jeevan Ladhe
| Attachment | Content-Type | Size |
|---|---|---|
| 0001-gzip-tap-test-remove-extra-variable.patch | application/octet-stream | 923 bytes |
| v10-0002-Add-a-LZ4-compression-method-for-server-side-compres.patch | application/octet-stream | 19.2 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Andres Freund | 2022-01-28 18:20:07 | Re: [PATCH] Disable bgworkers during servers start in pg_upgrade |
| Previous Message | Dagfinn Ilmari Mannsåker | 2022-01-28 17:35:33 | Re: Server-side base backup: why superuser, not pg_write_server_files? |