Re: Auxiliary Processes and MyAuxProc - Mailing list pgsql-hackers

From Mike Palmiotto
Subject Re: Auxiliary Processes and MyAuxProc
Date
Msg-id CAMN686G-J6Xv0N4jiL=SuJq412ZZK0raoDMeuv+KCf8s0bONRw@mail.gmail.com
Whole thread Raw
In response to Re: Auxiliary Processes and MyAuxProc  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: Auxiliary Processes and MyAuxProc  (Mike Palmiotto <mike.palmiotto@crunchydata.com>)
List pgsql-hackers
On Mon, Feb 25, 2019 at 1:29 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
>
> Mike Palmiotto <mike.palmiotto@crunchydata.com> writes:
> <snip>
>
> > For some context, I'm trying to come up with a patch to set the
> > process identifier (MyAuxProc/am_autovacuumworker/launcher,
> > am_archiver, etc.) immediately after forking,
>
> Don't we do that already?

Kind of. The indicators are set in the Main functions after
InitPostmasterChild and some other setup. In my little bit of digging
I found InitPostmasterChild to be the best place for a centralized
"early start-up hook." Is there a better place you can think of
off-hand?

> <snip>
>
> If memory serves, StartChildProcess already was an attempt to unify
> the treatment of postmaster children.  It's possible that another
> round of unification would be productive, but I think you'll find
> that there are random small differences in requirements that'd
> make it messy.

It kind of seemed like it, but I noticed the small differences in
requirements, which made me a bit hesitant. I'll go ahead and see what
I can do and submit the patch for consideration.

-- 
Mike Palmiotto
Software Engineer
Crunchy Data Solutions
https://crunchydata.com


pgsql-hackers by date:

Previous
From: Robert Haas
Date:
Subject: Re: POC: converting Lists into arrays
Next
From: David Steele
Date:
Subject: Re: Remove Deprecated Exclusive Backup Mode