|From:||Shay Rojansky <roji(at)roji(dot)org>|
|Subject:||[PATCH] Allow UNLISTEN during recovery|
|Views:||Raw Message | Whole Thread | Download mbox|
Here is a tiny patch removing PreventCommandDuringRecovery() for UNLISTEN.
See previous discussion in
In a nutshell, this prevents an error being raised when UNLISTEN is issued
during recovery. The operation is a no-op (since LISTEN is still
disallowed). This logic here is that some clients (namely Npgsql) issue
UNLISTEN * to clear connection state (in the connection pool), but this
needlessly breaks when the backend is in recovery.
On a related note, there currently doesn't seem to be a good way for
clients to know whether the backend is in recovery. As a backend can come
out of recovery at any point, perhaps an asynchronous ParameterStatus
announcing this state change could be useful.
Hopefully this also qualifies for backporting to earlier version branches.
|Next Message||Andrew Dunstan||2018-11-18 17:46:14||Re: _isnan() on Windows|
|Previous Message||Dmitry Dolgov||2018-11-18 11:18:33||Re: New GUC to sample log queries|