| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | Bruce Momjian <bruce(at)momjian(dot)us> |
| Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Jelte Fennema-Nio <postgres(at)jeltef(dot)nl>, Heikki Linnakangas <hlinnaka(at)iki(dot)fi>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Daniel Gustafsson <daniel(at)yesql(dot)se>, Joel Jacobson <joel(at)compiler(dot)org>, Andrew Dunstan <andrew(at)dunslane(dot)net>, Gabriele Bartolini <gabriele(dot)bartolini(at)enterprisedb(dot)com>, Magnus Hagander <magnus(dot)hagander(at)redpill-linpro(dot)com>, Maciek Sakrejda <m(dot)sakrejda(at)gmail(dot)com> |
| Subject: | Re: Possibility to disable `ALTER SYSTEM` |
| Date: | 2024-03-26 17:23:56 |
| Message-ID: | 2992935.1711473836@sss.pgh.pa.us |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Bruce Momjian <bruce(at)momjian(dot)us> writes:
> I am thinking "enable_alter_system_command" is probably good because we
> already use "enable" so why not reuse that idea, and I think "command"
> is needed because we need to clarify we are talking about the command,
> and not generic altering of the system. We could use
> "enable_sql_alter_system" if people want something shorter.
Robert already mentioned why not use "enable_": up to now that prefix
has only been applied to planner plan-type-enabling GUCs. I'd be okay
with "allow_alter_system_command", although I find it unnecessarily
verbose.
> Will people think this allows non-root users to use ALTER SYSTEM if
> enabled?
They'll soon find out differently, so I'm not concerned about that.
regards, tom lane
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Andrey M. Borodin | 2024-03-26 17:26:14 | Re: UUID v7 |
| Previous Message | Tom Lane | 2024-03-26 17:20:06 | Re: Propagate pathkeys from CTEs up to the outer query |