Re: Configuration Parameter/GUC value validation hook - Mailing list pgsql-hackers

From Tom Lane
Subject Re: Configuration Parameter/GUC value validation hook
Date
Msg-id 1845018.1651503285@sss.pgh.pa.us
Whole thread Raw
In response to Configuration Parameter/GUC value validation hook  (Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com>)
Responses Re: Configuration Parameter/GUC value validation hook
List pgsql-hackers
Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com> writes:
> I propose to add a simple new hook in set_config_option (void
> set_config_option_hook(struct config_generic *record);) and the
> vendors can implement their own platform-dependent extensions to
> accept or reject certain parameters (based on platform/VM
> configuration) that are of interest to them.

This seems entirely useless.  Vendors are unlikely to have any better
idea than we do about what are "reasonable" values.  Moreover, if they
did, modifying the source code directly would be an easier route to
introducing their code than making use of a hook (which'd require
finding a way to ensure that some extension is loaded).

In general, I think you need a much more concrete use-case than this
before proposing a new hook.  Otherwise we're going to have tons of
hooks that we don't know whether they're actually useful or being
used by anyone.

            regards, tom lane



pgsql-hackers by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: Proposal for internal Numeric to Uint64 conversion function.
Next
From: Pavel Stehule
Date:
Subject: Re: strange slow query - lost lot of time somewhere