> I think that what would actually be of some use nowadays is a LOG-level
> message emitted if the wraparound *isn't* activated immediately at start.
> But otherwise, we should follow the rule that silence is golden.
Concretely, how about the attached? It preserves the original
"protections are now enabled" message at LOG level, but emits it only
when oldestOffsetKnown becomes true *after* startup. Meanwhile, if
oldestOffsetKnown is still not true at the conclusion of TrimMultiXact,
then it emits a new LOG message about "protections are not active".
In this way we have LOG messages but they're only emitted in "interesting"
I dropped the IsUnderPostmaster test because I see no good reason not
to warn in standalone backends as well.
I think this might actually be a reasonable candidate to back-patch,
because a deficiency of the existing code is that it fails to warn
you when something's wrong. But in any case I'd like to put it in HEAD.
regards, tom lane
In response to
pgsql-hackers by date
|Next:||From: Peter Eisentraut||Date: 2017-03-10 23:57:00|
|Subject: Re: Need a builtin way to run all tests faster manner|
|Previous:||From: Corey Huinker||Date: 2017-03-10 23:19:25|
|Subject: Re: asynchronous execution|