From: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com> |
---|---|
To: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
Cc: | Marc Mamin <M(dot)Mamin(at)intershop(dot)de>, 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 13:14:05 |
Message-ID: | CAB7nPqS1v3XLLcnDDonoo79VfTfTgrao5PyoJPMQjXBVvOOS4A@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Sat, Jul 25, 2015 at 9:56 PM, Andrew Dunstan <andrew(at)dunslane(dot)net> wrote:
>
> On 07/25/2015 03:20 AM, Andrew Dunstan wrote:
>>
>>
>> On 07/25/2015 02:34 AM, Marc Mamin wrote:
>>>
>>>
>>>> Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
>>>>>
>>>>> Hmm. Yeah. It looks like commit
>>>>> a7ad5cf0cfcfab8418000d652fa4f0c6ad6c8911
>>>>> changed from using the default compression for libz to using the
>>>>> compression set in pg_dump options, which defaults to 0. This actually
>>>>> seems like the right thing to do, but it certainly should have been
>>>>> called out much more forcefully in release notes, and arguably should
>>>>> not have been changed in stable releases. Not sure what we do about it
>>>>> now.
>>>
>>> really 0? wouldn't that mean no compression at all?
>>
>>
>> No, that's not right either. The default should be Z_DEFAULT_COMPRESSION,
>> so we shouldn't actually see a difference in the default case. And it is
>> definitely compressing some. So I'm now puzzled by what you're seeing.
>>
>>
>
>
> OK, I have got this worked out. I'll have a bug fix shortly.
So you are basically planning to switch to Z_BEST_SPEED instead of
Z_DEFAULT_COMPRESSION where needed for the default code path?
--
Michael
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2015-07-25 13:54:15 | Re: Supporting TAP tests with MSVC and Windows |
Previous Message | Andrew Dunstan | 2015-07-25 12:56:02 | Re: pg_dump -Fd and compression level |