From: | Magnus Hagander <magnus(at)hagander(dot)net> |
---|---|
To: | Simon Riggs <simon(at)2ndquadrant(dot)com> |
Cc: | Magnus Hagander <mha(at)postgresql(dot)org>, pgsql-committers(at)postgresql(dot)org |
Subject: | Re: pgsql: Report which WAL sync method we are trying to change *to* when it |
Date: | 2008-05-12 19:06:30 |
Message-ID: | 20080512210630.050875a7@mha-laptop.hagander.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Simon Riggs wrote:
> On Mon, 2008-05-12 at 14:27 +0000, Magnus Hagander wrote:
> > Log Message:
> > -----------
> > Report which WAL sync method we are trying to change *to* when it
> > fails, not which one we had before (that worked, and thus is
> > completley irrelevant)
>
> Interesting perspective.
>
> If it breaks, I'd rather be able to put it back the way it was than
> regret in technicolour that my new choice was a bad one. ;-)
Well, the message itself indicated that it was the new one...
> Not everybody keeps a change log.
>
> Could we report both?
Yes, we could easily do that if we want to.
But - this is not the error you get when you try to set it. It's the
error you get when you try to *use* it. And really, it's a "should
never happen" error. (The reason it happens this time is due to another
bug). So I don't think doing so would actually help your case - it's
already covered elsewhere in the code where we'll rollback the setting
when you try to change it instead of PANICing.
//Magnus
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2008-05-12 19:26:03 | Re: pgsql: Report which WAL sync method we are trying to change *to* when it |
Previous Message | Simon Riggs | 2008-05-12 19:04:25 | Re: pgsql: Report which WAL sync method we are trying to change *to* when it |