Hi Andres, (Not my intention to miss your message, I have just noticed it.)
On Wed, Aug 08, 2018 at 01:41:27AM -0700, Andres Freund wrote: > I can't parse this. "Even if this is an atomic operation, this can be > safely done lock-less" - that seems like a contradictory sentence. Is > there a "not" missing? Yes, a "not" has gone missing here. I reworked the comment block as mentioned upthread. > Also, this seems like insufficient reasoning. What guarantees the > visibility of the flag? You're going to have to talk about externally > implied memory ordering here. Or add explicit barriers - the latter is > probably preferrable. Well, we use BackendIdGetProc() in this case, where we could finish with information out-of-date pretty quickly, and there is no special reasoning for backendId and databaseId for autovacuum but... Perhaps you could explain more what you have in mind? And it is not like this relies on the number of elements in PGPROC. -- Michael
signature.asc
Description: PGP signature