| From: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
|---|---|
| To: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
| Cc: | Stephen Frost <sfrost(at)snowman(dot)net>, 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 04:56:29 |
| Message-ID: | 20130820045629.GB6564@eldon.alvh.no-ip.org |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Amit Kapila escribió:
> 3. postgresql.conf will contain include directive in below form:
> #include = 'postgresql.auto.conf'
> Whenever user wants to use Alter System, he needs to enable it
> after first time using ALTER SYSTEM.
This seems wrong to me. If the auto file is read by an include line in
postgresql.conf, what is its priority w.r.t. files placed in an
hypothetical conf.d directory? Hopefully snippets put in conf.d/ by
puppet/chef will override the settings in postgresql.conf (i.e. conf.d/
should be processed after postgresql.conf, not before); and hopefully
ALTER SYSTEM will in turn override conf.d. I see no way to have ALTER
SYSTEM handled by an include line, yet still have it override conf.d.
If we want to make ALTER SYSTEM disable-able from postgresql.conf, I
think it should be an explicit option, something like
enable_alter_system = on
or something like that.
--
Álvaro Herrera http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Boszormenyi Zoltan | 2013-08-20 05:18:45 | Re: UNNEST with multiple args, and TABLE with multiple funcs |
| Previous Message | Alvaro Herrera | 2013-08-20 04:45:44 | Re: danger of stats_temp_directory = /dev/shm |