Re: [Patch] Create a new session in postmaster by calling setsid() - Mailing list pgsql-hackers

From Paul Guo
Subject Re: [Patch] Create a new session in postmaster by calling setsid()
Date
Msg-id CAEET0ZFy74SA-mrYN4pG76zujg__H2UKeNN-8ETX4-gnj5uwhQ@mail.gmail.com
Whole thread Raw
In response to Re: [Patch] Create a new session in postmaster by calling setsid()  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: [Patch] Create a new session in postmaster by calling setsid()
List pgsql-hackers


On Thu, Aug 2, 2018 at 10:30 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
Paul Guo <pguo@pivotal.io> writes:
> [ make the postmaster execute setsid() too ]

I'm a bit skeptical of this proposal.  Forcing the postmaster to
dissociate from its controlling terminal is a good thing in some
scenarios, but probably less good in others, and manual postmaster
starts are probably mostly in the latter class.

I wonder whether having "pg_ctl start" do a setsid() would accomplish
the same result with less chance of breaking debugging usages.

                        regards, tom lane

Yes, if considering the case of starting postmaster manually, we can not create
a new session in postmaster, so pg_ctl seems to be a good place for setsid()
call. Attached a newer patch. Thanks.


Attachment

pgsql-hackers by date:

Previous
From: Craig Ringer
Date:
Subject: Allow postgres_fdw passwordless non-superuser conns with priorsuperuser permission
Next
From: Ashutosh Bapat
Date:
Subject: Re: TupleTableSlot abstraction