From: | Fabrízio de Royes Mello <fabriziomello(at)gmail(dot)com> |
---|---|
To: | Robins Tharakan <tharakan(at)gmail(dot)com> |
Cc: | "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>, David Fetter <david(at)fetter(dot)org>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Robert Haas <robertmhaas(at)gmail(dot)com>, "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com>, Stephen Frost <sfrost(at)snowman(dot)net>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com> |
Subject: | Re: Patch: Add --no-comments to skip COMMENTs with pg_dump |
Date: | 2017-08-07 15:14:52 |
Message-ID: | CAFcNs+q6pvBs1r0zv6j4ez0qe3K3pDe_OFrMOpGunB5f4ajrvQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, Aug 7, 2017 at 10:43 AM, Robins Tharakan <tharakan(at)gmail(dot)com> wrote:
>
> On 20 July 2017 at 05:14, Robins Tharakan <tharakan(at)gmail(dot)com> wrote:
>>
>> On 20 July 2017 at 05:08, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
wrote:
>>>
>>> On Wed, Jul 19, 2017 at 8:59 PM,
>>> Fabrízio de Royes Mello
>>> > You should add the properly sgml docs for this pg_dumpall change also.
>>>
>>> Tests of pg_dump go to src/bin/pg_dump/t/ and tests for objects in
>>> extensions are in src/test/modules/test_pg_dump, but you just care
>>> about the former with this patch. And if you implement some new tests,
>>> look at the other tests and base your work on that.
>>
>>
>> Thanks Michael /
>> Fabrízio.
>>
>> Updated patch (attached) additionally adds SGML changes for pg_dumpall.
>> (I'll try to work on the tests, but sending this
>> nonetheless
>> ).
>>
>
> Attached is an updated patch (v4) with basic tests for pg_dump /
pg_dumpall.
> (Have zipped it since patch size jumped to ~40kb).
>
The patch applies cleanly to current master and all tests run without
failures.
I also test against all current supported versions (9.2 ... 9.6) and didn't
find any issue.
Changed status to "ready for commiter".
Regards,
--
Fabrízio de Royes Mello
Consultoria/Coaching PostgreSQL
>> Timbira: http://www.timbira.com.br
>> Blog: http://fabriziomello.github.io
>> Linkedin: http://br.linkedin.com/in/fabriziomello
>> Twitter: http://twitter.com/fabriziomello
>> Github: http://github.com/fabriziomello
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2017-08-07 15:17:20 | Re: Crash report for some ICU-52 (debian8) COLLATE and work_mem values |
Previous Message | Daniel Verite | 2017-08-07 13:52:33 | Re: Crash report for some ICU-52 (debian8) COLLATE and work_mem values |