From: | Daniel Gustafsson <daniel(at)yesql(dot)se> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | James Doherty <jim(at)jdoherty(dot)net>, pgsql-docs(at)lists(dot)postgresql(dot)org |
Subject: | Re: Another pg_dump using split and gzip for large databases |
Date: | 2021-07-29 19:12:40 |
Message-ID: | 168C3060-BD8F-4C34-91F4-5F2F52273631@yesql.se |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-docs |
> On 29 Jul 2021, at 21:10, Bruce Momjian <bruce(at)momjian(dot)us> wrote:
>
> On Thu, Jul 29, 2021 at 07:15:21PM +0200, Daniel Gustafsson wrote:
>>> On 29 Jul 2021, at 17:29, Bruce Momjian <bruce(at)momjian(dot)us> wrote:
>>>
>>> On Thu, Jul 29, 2021 at 05:23:20PM +0200, Daniel Gustafsson wrote:
>>>>> On 29 Jul 2021, at 16:58, Bruce Momjian <bruce(at)momjian(dot)us> wrote:
>>>>
>>>>> OK, I can adjust it to say "GNU split" or just skip this addition.
>>>>
>>>> I don't have strong feelings either way, with a slight preference for writing
>>>> it as "GNU split" since it's still a good tip for a large amount of readers.
>>>
>>> Yes, I am on the fence for the same reasons but lean toward moving
>>> forward for your same reasons.
>>
>> Making it two separate examples is a good idea, tipping the scale to a +1 from me.
>>
>> + If using <application>GNU split</application>, it is possible to
>> A small nitpick is that we write this as GNU <application>..</application>
>> elsewhere on this page (like GNU tar for example).
>
> OK, updated patch attached. FYI, I put the GNU inside the application
> name in the first version because we specifically only want the GNU
> version of that application, but I think taking it out for consistency
> is fine.
LGTM
--
Daniel Gustafsson https://vmware.com/
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2021-07-29 22:24:17 | Re: getting table name from partition |
Previous Message | Bruce Momjian | 2021-07-29 19:10:34 | Re: Another pg_dump using split and gzip for large databases |