From: | Magnus Hagander <magnus(at)hagander(dot)net> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Reversed sync check in pg_receivewal |
Date: | 2017-04-11 13:38:07 |
Message-ID: | CABUevExMz0xmhmVKU8ZU-J8OUUOVGGZ8X-G8UXBK2NOuj+-2kQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Tue, Apr 11, 2017 at 3:19 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Magnus Hagander <magnus(at)hagander(dot)net> writes:
> > Attached patch reverses the check, and adds a failure message. I'd
> > appreciate a quick review in case I have the logic backwards in my
> head...
>
> I think the patch is correct, but if there's any documentation of the
> walmethod APIs that would allow one to assert which side of the API got
> this wrong, I sure don't see it. Would it be unreasonable to insist
> on some documentation around that?
>
>
Agreed.
Would you say comments in the struct in walmethods.h is enough, or were you
thinking actual sgml docs when you commented that?
--
Magnus Hagander
Me: https://www.hagander.net/ <http://www.hagander.net/>
Work: https://www.redpill-linpro.com/ <http://www.redpill-linpro.com/>
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2017-04-11 13:40:12 | Re: Patch: Write Amplification Reduction Method (WARM) |
Previous Message | Aleksander Alekseev | 2017-04-11 13:28:09 | Re: Reversed sync check in pg_receivewal |