pg.ddx.io pgsql-hackers@postgresql.org mailing list archivehelp / color / mirror / Atom feed
From: Nathan Bossart <nathandbossart@gmail.com>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Robert Haas <robertmhaas@gmail.com>
Cc: Andres Freund <andres@anarazel.de>
Cc: Julien Rouhaud <rjuju123@gmail.com>
Cc: Michael Paquier <michael@paquier.xyz>
Cc: Bossart, Nathan <bossartn@amazon.com>
Cc: Fujii Masao <masao.fujii@oss.nttdata.com>
Cc: wangsh.fnst@fujitsu.com <wangsh.fnst@fujitsu.com>
Cc: Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com>
Cc: Greg Sabino Mullane <htamfids@gmail.com>
Cc: pgsql-hackers@lists.postgresql.org <pgsql-hackers@lists.postgresql.org>
Subject: Re: make MaxBackends available in _PG_init
Date: Wed, 13 Apr 2022 11:30:40 -0700
Message-ID: <20220413183040.GA2116671@nathanxps13> (raw)
In-Reply-To: <1102589.1649865908@sss.pgh.pa.us>
References: <995617.1649799996@sss.pgh.pa.us>
<20220412222616.GA2113503@nathanxps13>
<CA+TgmoYA09kL2nTKiONj6pCcZ7KNX28YUyseKODJq+trNAXunw@mail.gmail.com>
<1088297.1649856441@sss.pgh.pa.us>
<CA+TgmoaPC_bpLTfZ8uVuYoV+QNkYztp2BbFNHi_S+VHoChFSpQ@mail.gmail.com>
<1095139.1649860995@sss.pgh.pa.us>
<CA+TgmoaN0zj2Km-ZLWnCL_W_x7rrULebDi0yoK=mGyf6yC_cqw@mail.gmail.com>
<1100207.1649864342@sss.pgh.pa.us>
<CA+TgmobVwPLaEq9UrKtnaoQiA+2LQg+NS8ANhXNGNBf+c_yvRw@mail.gmail.com>
<1102589.1649865908@sss.pgh.pa.us>
On Wed, Apr 13, 2022 at 12:05:08PM -0400, Tom Lane wrote:
> Robert Haas <robertmhaas@gmail.com> writes:
>> It may be too much to hope that we're going to completely get rid of
>> process_shared_preload_libraries_in_progress tests.
>
> Perhaps, but this is definitely an area that could stand to have some
> serious design thought put into it. You're quite right that it's
> largely a mess today, but I think proposals like "forbid setting GUCs
> during _PG_init" would add to the mess rather than clean things up.
+1. The simplest design might be to just make a separate preload hook.
_PG_init() would install hooks, and then this preload hook would do
anything that needs to happen when the library is loaded via s_p_l.
However, I think we still want a new shmem request hook where MaxBackends
is initialized.
If we do move forward with the shmem request hook, do we want to disallow
shmem requests anywhere else, or should we just leave it up to extension
authors to do the right thing? Many shmem requests in _PG_init() are
probably okay unless they depend on MaxBackends or another GUC that someone
might change. Given that, I think I currently prefer the latter (option B
from upthread).
--
Nathan Bossart
Amazon Web Services: https://aws.amazon.com
view thread (132+ messages) latest in thread
Message-ID: <20220413183040.GA2116671@nathanxps13> Permalink: ../20220413183040.GA2116671@nathanxps13/ Also on: postgresql.org/message-id/20220413183040.GA2116671@nathanxps13 ·
reply
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Reply to all the recipients using the --to and --cc options: reply via email To: pgsql-hackers@postgresql.org Cc: nathandbossart@gmail.com, tgl@sss.pgh.pa.us, robertmhaas@gmail.com, andres@anarazel.de, rjuju123@gmail.com, michael@paquier.xyz, bossartn@amazon.com, masao.fujii@oss.nttdata.com, wangsh.fnst@fujitsu.com, bharath.rupireddyforpostgres@gmail.com, htamfids@gmail.com, pgsql-hackers@lists.postgresql.org Subject: Re: make MaxBackends available in _PG_init In-Reply-To: <20220413183040.GA2116671@nathanxps13> * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions for how to clone and mirror all data and code used for this inbox