Re: should we document an example to set multiple libraries in shared_preload_libraries? - Mailing list pgsql-hackers

From Bharath Rupireddy
Subject Re: should we document an example to set multiple libraries in shared_preload_libraries?
Date
Msg-id CALj2ACVVoFctnh0vnzaF94gHfSYnOqM=1K1wu64veU+7+p-s=w@mail.gmail.com
Whole thread Raw
In response to Re: should we document an example to set multiple libraries in shared_preload_libraries?  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: should we document an example to set multiple libraries in shared_preload_libraries?
List pgsql-hackers
On Wed, Dec 1, 2021 at 6:45 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
>
> Justin Pryzby <pryzby@telsasoft.com> writes:
> > +1 to document it, but it seems like the worse problem is allowing the admin to
> > write a configuration which causes the server to fail to start, without having
> > issued a warning.
>
> > I think you could fix that with a GUC check hook to emit a warning.
> > I'm not sure what objections people might have to this.  Maybe it's confusing
> > to execute preliminary verification of the library by calling stat() but not do
> > stronger verification for other reasons the library might fail to load.  Like
> > it doesn't have the right magic number, or it's built for the wrong server
> > version.  Should factor out the logic from internal_load_library and check
> > those too ?
>
> Considering the vanishingly small number of actual complaints we've
> seen about this, that sounds ridiculously over-engineered.
> A documentation example should be sufficient.

Thanks. Here's the v1 patch adding examples in the documentation.

Regards,
Bharath Rupireddy.

Attachment

pgsql-hackers by date:

Previous
From: Amit Kapila
Date:
Subject: Re: Non-superuser subscription owners
Next
From: Bharath Rupireddy
Date:
Subject: Is it worth adding ReplicationSlot active_pid to ReplicationSlotPersistentData?