From: | Marc Mamin <M(dot)Mamin(at)intershop(dot)de> |
---|---|
To: | Andrew Dunstan <andrew(at)dunslane(dot)net>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: pg_dump -Fd and compression level |
Date: | 2015-07-25 19:07:13 |
Message-ID: | B6F6FD62F2624C4C9916AC0175D56D8828C1827B@jenmbs01.ad.intershop.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
>
>>
>> I propose to tighten pg_dump's rules so that only 0..9 are accepted as
>> arguments for -Z, and in compress_io.c:cfopen(), if compression is
>> equal to Z_DEFAULT_COMPRESSION, not add any explicit compression value
>> to the mode, thus using the zlib default.
>>
>>
>
>
>As per attached patch.
>
>Comments?
It seems that the first test on the compression in pg_backup_tar.c is now obsolete.
It didn't make much sense anyway.
211 if (AH->compression < 0 || AH->compression > 9)
212 AH->compression = Z_DEFAULT_COMPRESSION;
213
214 /* Don't compress into tar files unless asked to do so */
215 if (AH->compression == Z_DEFAULT_COMPRESSION)
216 AH->compression = 0;
217
218 /*
219 * We don't support compression because reading the files back is not
220 * possible since gzdopen uses buffered IO which totally screws file
221 * positioning.
222 */
223 if (AH->compression != 0)
224 exit_horribly(modulename,
225 "compression is not supported by tar archive format\n");
226 }
regards,
Marc
>
>cheers
>
>andrew
From | Date | Subject | |
---|---|---|---|
Next Message | Pavel Stehule | 2015-07-25 19:21:24 | Re: polymorphic types - enforce casting to most common type automatically |
Previous Message | Tom Lane | 2015-07-25 18:45:24 | Re: pg_dump -Fd and compression level |