From: | Christoph Berg <myon(at)debian(dot)org> |
---|---|
To: | Eren Başak <eren(at)citusdata(dot)com> |
Cc: | Daniel Gustafsson <daniel(at)yesql(dot)se>, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, Yugo Nagata <nagata(at)sraoss(dot)co(dot)jp>, Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, PostgreSQL mailing lists <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [HACKERS] Optional message to user when terminating/cancelling backend |
Date: | 2018-03-20 12:41:42 |
Message-ID: | 20180320124142.GC32205@msg.df7cb.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Re: Eren Başak 2018-03-20 <CAFNTstPcstV8Brqkg00a84V72b_FfnLinhu2C2Top+QssmwFhg(at)mail(dot)gmail(dot)com>
> Another thing is that, in a similar manner, we could allow changing the
> error code which might be useful for extensions. For example, Citus could
> use it to cancel remote backends when it detects a distributed deadlock and
> changes the error code to something retryable while doing so.
Another useful thing to do on top of this patch would be to include
messages when the termination comes from postgres itself, e.g. on a
server shutdown. Possibly, the message for pg_terminate_backend() itself could
say that someone invoke that, unless overridden.
FATAL: 57P01: terminating connection due to administrator command: server shutting down
FATAL: 57P01: terminating connection due to administrator command: restarting because of a crash of another server process
FATAL: 57P01: terminating connection due to administrator command: terminated by pg_terminate_backend()
Christoph
From | Date | Subject | |
---|---|---|---|
Next Message | Tels | 2018-03-20 12:56:25 | Re: [HACKERS] plpgsql - additional extra checks |
Previous Message | David Rowley | 2018-03-20 12:41:02 | Re: [HACKERS] path toward faster partition pruning |