Bharath Rupireddy <bharath.rupireddyforpostg...@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