From: | Magnus Hagander <magnus(at)hagander(dot)net> |
---|---|
To: | Daniel Gustafsson <daniel(at)yesql(dot)se> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Jelte Fennema-Nio <postgres(at)jeltef(dot)nl>, Bruce Momjian <bruce(at)momjian(dot)us>, Maciek Sakrejda <m(dot)sakrejda(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Joel Jacobson <joel(at)compiler(dot)org>, Andrew Dunstan <andrew(at)dunslane(dot)net>, Gabriele Bartolini <gabriele(dot)bartolini(at)enterprisedb(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Possibility to disable `ALTER SYSTEM` |
Date: | 2024-03-18 15:46:33 |
Message-ID: | CABUevEz_FLX-xw9pWxO_wyCi_=v2S8LwbqX5aOiB6Ja50MwUNw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, Mar 18, 2024 at 4:44 PM Daniel Gustafsson <daniel(at)yesql(dot)se> wrote:
>
> > On 18 Mar 2024, at 16:34, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
> >
> > On Mon, Mar 18, 2024 at 2:09 PM Daniel Gustafsson <daniel(at)yesql(dot)se> wrote:
> >>
> >>> On 18 Mar 2024, at 13:57, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
> >>
> >>> my proposal is something like this, taking a
> >>> bunch of text from Jelte's patch and some inspiration from Magnus's
> >>> earlier remarks:
> >>
> >> I still think any wording should clearly mention that settings in the file are
> >> still applied. The proposed wording says to implicitly but to avoid confusion
> >> I think it should be explicit.
> >
> > I haven't kept up with the thread, but in general I'd prefer it to
> > actually turn off parsing the file as well. I think just turning off
> > the ability to change it -- including the ability to *revert* changes
> > that were made to it before -- is going to be confusing.
>
> Wouldn't that break pgBackrest which IIRC write to .auto.conf directly
> without using ALTER SYSTEM?
Ugh of course. And not only that, it would also break pg_basebackup
which does the same.
So I guess that's not a good idea. I guess nobody anticipated this
when that was done:)
> > But, if we have decided it shouldn't do that, then IMHO we should
> > consider naming it maybe enable_alter_system_command instead -- since
> > we're only disabling the alter system command, not the actual feature
> > in total.
>
> Good point.
--
Magnus Hagander
Me: https://www.hagander.net/
Work: https://www.redpill-linpro.com/
From | Date | Subject | |
---|---|---|---|
Next Message | Ashutosh Bapat | 2024-03-18 15:47:31 | Re: Memory consumed by child SpecialJoinInfo in partitionwise join planning |
Previous Message | Daniel Gustafsson | 2024-03-18 15:44:20 | Re: Possibility to disable `ALTER SYSTEM` |