Re: pg_recvlogical prints bogus error when interrupted

Hi,

On 2022-10-24 08:15:11 +0530, Bharath Rupireddy wrote:
> I came up with the attached v2 patch, please have a look.

Thanks for working on this!


> +		/* When we get SIGINT/SIGTERM, we exit */
> +		if (ready_to_exit)
> +		{
> +			/*
> +			 * Try informing the server about our exit, but don't wait around
> +			 * or retry on failure.
> +			 */
> +			(void) PQputCopyEnd(conn, NULL);
> +			(void) PQflush(conn);
> +			time_to_abort = ready_to_exit;

This doesn't strike me as great - because the ready_to_exit isn't checked in
the loop around StreamLogicalLog(), we'll reconnect if something else causes
StreamLogicalLog() to return.

Why do we need both time_to_abort and ready_to_exit? Perhaps worth noting that
time_to_abort is still an sig_atomic_t, but isn't modified in a signal
handler, which seems a bit unnecessarily confusing.

Greetings,

Andres Freund