Re: pg_dump -Fd and compression level - Mailing list pgsql-hackers

From Andrew Dunstan
Subject Re: pg_dump -Fd and compression level
Date
Msg-id 55B77956.10000@dunslane.net
Whole thread Raw
In response to Re: pg_dump -Fd and compression level  (Marc Mamin <M.Mamin@intershop.de>)
List pgsql-hackers
On 07/27/2015 03:52 AM, Marc Mamin wrote:
>>>> 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     }
>>>     
>>>     
>> In fact, the first two tests look unnecessary. Neither condition should
>> be possible now.
>>
> Hello,
>
> Isn't the second test still required if you call pg_dump -Ft without setting -Z0 explicitly ?
> (=> AH->compression == Z_DEFAULT_COMPRESSION)



No. Z_DEFAULT_COMPRESSION is only set for directory and custom archive 
types. See pg_dump.c at lines 578-592.


cheers

andrew




pgsql-hackers by date:

Previous
From: Kouhei Kaigai
Date:
Subject: Re: [DESIGN] ParallelAppend
Next
From: Marc Mamin
Date:
Subject: Re: proposal: multiple psql option -c