Re: pg_receivewal documentation

From: Jesper Pedersen <jesper(dot)pedersen(at)redhat(dot)com>
To: Laurenz Albe <laurenz(dot)albe(at)cybertec(dot)at>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: pg_receivewal documentation
Date: 2019-07-09 17:18:52
Message-ID: e742f5d0-c8cf-d404-2e59-f51bd9b64760@redhat.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi Laurenz,

On 7/9/19 5:16 AM, Laurenz Albe wrote:
> On Thu, 2019-06-27 at 10:06 -0400, Jesper Pedersen wrote:
>> Here is a patch for the pg_receivewal documentation to highlight that
>> WAL isn't acknowledged to be applied.
>
> I think it is a good idea to document this, but I have a few quibbles
> with the patch as it is:
>
> - I think there shouldn't be commas after the "note" and before the "if".
> Disclaimer: I am not a native speaker, so I am lacking authority.
>
> - The assertion is wrong. "on" (remote flush) is perfectly fine
> for synchronous_commit, only "remote_apply" is a problem.
>
> - There is already something about "--synchronous" in the "Description"
> section. It might make sense to add the additional information there.
>
> How about the attached patch?
>

Thanks for the review, and the changes.

However, I think it belongs in the --synchronous section, so what about
moving it there as attached ?

Best regards,
Jesper

Attachment Content-Type Size
v3_pgreceivewal-doc.patch text/x-patch 1.2 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tomas Vondra 2019-07-09 17:49:44 Re: Increasing default value for effective_io_concurrency?
Previous Message Tomas Vondra 2019-07-09 16:59:15 Re: [PATCH] Fix trigger argument propagation to child partitions