Mark Dilger <mark.dil...@enterprisedb.com> writes:
> # Running: pg_ctl kill QUIT 2083
> ok 4 - killed process with SIGQUIT
> # pump_until: process terminated unexpectedly when searching for
> "(?^m:WARNING: terminating connection because of crash of another server
> process|server closed the connection unexpectedly|connection to server was
> lost)" with stream: "psql:<stdin>:9: WARNING: terminating connection because
> of unexpected SIGQUIT signal
> # psql:<stdin>:9: could not send data to server: Socket is not connected
> # "
> not ok 5 - psql query died successfully after SIGQUIT
And there we have it: the test wasn't updated for the new backend message
spelling, and we're seeing a different frontend behavior. Evidently the
backend is dying before we're able to send the "SELECT 1;" to it.
I'm not quite sure whether it's a libpq bug that it doesn't produce the
"connection to server was lost" message here, but in any case I suspect
that we shouldn't be checking for the second and third regex alternatives.
The "terminating connection" warning absolutely should get through, and
if it doesn't we want to know about it. So my proposal for a fix is
to change the regex to be just "WARNING: terminating connection because
of unexpected SIGQUIT signal".
regards, tom lane
