Re: termination of backend waiting for sync rep generates a junk log message

From: "Kevin Grittner" <Kevin(dot)Grittner(at)wicourts(dot)gov>
To: "Robert Haas" <robertmhaas(at)gmail(dot)com>, "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: "Simon Riggs" <simon(at)2ndquadrant(dot)com>, "Fujii Masao" <masao(dot)fujii(at)gmail(dot)com>, "PostgreSQL-development" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: termination of backend waiting for sync rep generates a junk log message
Date: 2011-10-24 17:23:40
Message-ID: 4EA558CC02000025000424CE@gw.wicourts.gov
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:

> It might be that it'd work to deal with this by reducing the
> reported strength of all such cases from ERROR to WARNING. Not
> sure that that's a good idea, but it might work.

Throwing an error on commit of a transaction when its work has
actually been persisted seems as bad as failing to persist the work
of a transaction when the commit appears successful. If we know we
have met the persistence guarantees and some problem occurs after
that, then it seems to me we should issue a warning and indicate
success. If we don't know whether the work will be persisted, it
seems to me we can block until we find out (if that's feasible) or
break the connection without indicating one way or the other. The
work of the transaction must either all be persisted or not (I'm
assuming this isn't a problem here), and any user must be prepared
to deal with a broken connection before response on a commit
attempt.

I don't know whether changing all failures beyond the persistence
point in a commit to warnings is sufficient, but I think it is
required for anything which doesn't break the connection.

-Kevin

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2011-10-24 17:29:43 Re: [PATCH] Use new oom_score_adj without a new compile-time constant
Previous Message Jan Urbański 2011-10-24 17:18:04 Re: Monster query