| From: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com> |
|---|---|
| To: | Craig Ringer <craig(at)2ndquadrant(dot)com> |
| Cc: | PostgreSQL mailing lists <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: pg_receivewal and messages printed in non-verbose mode |
| Date: | 2017-06-14 04:04:17 |
| Message-ID: | CAB7nPqRL2xSZv+Zs+OkNrPxUSG7r_SCvBXhsNFAB_pZAUtuP8Q@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Tue, Jun 13, 2017 at 4:50 PM, Craig Ringer <craig(at)2ndquadrant(dot)com> wrote:
> On 13 June 2017 at 14:33, Michael Paquier <michael(dot)paquier(at)gmail(dot)com> wrote:
>> Those come from stop_streaming in pg_receivewal.c. Shouldn't those
>> messages only show up to the user if --verbose is used? It seems
>> strange to me that at least the first one is written to the user as
>> that's not an error after promoting a standby.
>
> I agree. At least the first should be --verbose only.
I have been looking at all the code surrounding pg_receivewal and
pg_recvlogical and those are indeed the two only places where we print
a message in non-verbose mode even if those are not explicit errors.
pg_recvlogical does not show up any messages when it is signaled or
when it receives SIGINT or reaches the end of LSN position. I don't
think that this is worth complicating the code for, just noticed the
inconsistency on the way.
Perhaps a committer will care about that. Or not. For now I am just
adding that in the CF.
--
Michael
| Attachment | Content-Type | Size |
|---|---|---|
| receivewal-verbose-fix.patch | application/octet-stream | 990 bytes |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Pavel Stehule | 2017-06-14 04:05:24 | Re: v10beta pg_catalog diagrams |
| Previous Message | Amit Langote | 2017-06-14 03:59:49 | Re: outfuncs.c utility statement support |