| From: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
|---|---|
| To: | Stephen Frost <sfrost(at)snowman(dot)net> |
| Cc: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, Dimitri Fontaine <dimitri(at)2ndquadrant(dot)fr>, Andres Freund <andres(at)2ndquadrant(dot)com>, Josh Berkus <josh(at)agliodbs(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Fujii Masao <masao(dot)fujii(at)gmail(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: ALTER SYSTEM SET command to change postgresql.conf parameters (RE: Proposal for Allow postgresql.conf values to be changed via SQL [review]) |
| Date: | 2013-08-20 02:51:49 |
| Message-ID: | CAA4eK1+--qmfi5LzDBjsRNjTd4PKj1zrbt9+ED0fxsEB1BZA_w@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Tue, Aug 20, 2013 at 5:49 AM, Stephen Frost <sfrost(at)snowman(dot)net> wrote:
> * Amit Kapila (amit(dot)kapila16(at)gmail(dot)com) wrote:
>> I think disabling ALTER SYSTEM can be more easily supported if we
>> choose one of below options:
>> a. UI into contrib module, people who don't want doesn't include it
>> b. Have an include file mechanism, so that user can comment the
>> include in postgresql.conf and disable it.
>
> Either of these would be good options, imv. In my new world order, it'd
> be a '#include' in the postgresql.conf- which is commented out by
> default.
If both are okay, then I would like to go with second option (include
file mechanism).
I think by default, we should allow auto file to be included and if
user faces any problem or otherwise,
then he can decide to disable it.
With Regards,
Amit Kapila.
EnterpriseDB: http://www.enterprisedb.com
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Stephen Frost | 2013-08-20 02:57:28 | Re: ALTER SYSTEM SET command to change postgresql.conf parameters (RE: Proposal for Allow postgresql.conf values to be changed via SQL [review]) |
| Previous Message | Fabrízio de Royes Mello | 2013-08-20 02:30:21 | Re: Re: Patch to add support of "IF NOT EXISTS" to others "CREATE" statements |