Horton

Re: Disabling ALTER SYSTEM SET WAS: Re: ALTER SYSTEM SET command to change postgresql.conf parameters

Bruce Momjian <bruce@momjian.us>

View thread View on postgr.es
From: Bruce Momjian <bruce@momjian.us>
To: Stephen Frost <sfrost@snowman.net>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, cedric@2ndquadrant.com, pgsql-hackers@postgresql.org, Stefan Kaltenbrunner <stefan@kaltenbrunner.cc>, Dimitri Fontaine <dimitri@2ndquadrant.fr>, Josh Berkus <josh@agliodbs.com>, Andres Freund <andres@2ndquadrant.com>, Greg Stark <stark@mit.edu>, Alvaro Herrera <alvherre@2ndquadrant.com>, Fujii Masao <masao.fujii@gmail.com>, Robert Haas <robertmhaas@gmail.com>, Amit Kapila <amit.kapila@huawei.com>
Date: 2013-08-06T15:08:21Z
Lists: pgsql-hackers 
On Tue, Aug  6, 2013 at 10:58:52AM -0400, Stephen Frost wrote:
> * Tom Lane (tgl@sss.pgh.pa.us) wrote:
> > =?iso-8859-1?q?C=E9dric_Villemain?= <cedric@2ndquadrant.com> writes:
> > > Maybe better to provide a contrib/ to modify config, then design what we can 
> > > achieve more with an ALTER SYSTEM command.
> > 
> > Hmm ... putting the UI into a contrib module would neatly solve the
> > problem for people who don't want the functionality.  They just don't
> > install the module.  Another thought here is that it seems like we've
> > been discussing multiple scenarios that don't necessarily all have the
> > same best solution.  Maybe we should think in terms of multiple contrib
> > modules.
> 
> I'd certainly be happier with that- but it already exists, to some
> extent, in the form of the pgAdmin "adminpack" contrib module.  Perhaps
> we should be working on improving what we've already got there rather
> than adding something completely new..

Yes, "adminpack" has mostly read/write file I/O operations --- it would
need a higher-level API.

-- 
  Bruce Momjian  <bruce@momjian.us>        http://momjian.us
  EnterpriseDB                             http://enterprisedb.com

  + It's impossible for everything to be true. +