On Thu, Dec 20, 2007 at 04:47:33AM -0800, Trevor Talbot wrote:
> On 12/20/07, Magnus Hagander <magnus@hagander.net> wrote:
> > On Thu, Dec 20, 2007 at 04:39:55AM -0800, Trevor Talbot wrote:
> > > On 12/20/07, Magnus Hagander <magnus@hagander.net> wrote:
>
> > > > ereport(WARNING,
> > > > (errmsg("could not open file \"%s\": %s violation", fileName,
> > > > (GetLastError() ==
> > > > ERROR_SHARING_VIOLATION)?_("sharing"):_("lock")),
> > > > errdetail("Continuing to retry for 30 seconds."),
> > > > errhint("You may have antivirus, backup or similar software
> > > > interfering with the database.")));
>
> > > Without looking myself, is it possible for errhint() or errdetail() to
> > > do something that affects GetLastError()? It's like errno, checking it
> > > very far away from the call site makes me nervous.
>
> > I guess, but it shouldn't matter. We're giong to loop right back up and do a
> > new CreateFile() after this, which will overwrite it again.
>
> I mean for the purposes of the report. I'm worried the message might
> say it's a lock violation when it's really a sharing violation.
Wouldn't we then have the same problem for every place that does a %i and
report GetLastError() or errno? And we have a *lot* of those... I would've
thought the framework thinks of that, but I haven't actually verified that.
If it's a problem, it's trivial to save the value off in a variable, of
course.
//Magnus
