| From: | Dmitry Dolgov <9erthalion6(at)gmail(dot)com> |
|---|---|
| To: | Michael Banck <mbanck(at)gmx(dot)net> |
| Cc: | Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com>, Yuhang Qiu <iamqyh(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: [PATCH] Add ALTER SYSTEM RELOAD |
| Date: | 2026-09-24 15:34:56 |
| Message-ID: | arVCaX6CTKco9DEU@ddolgov-thinkpadt14sgen1.rmtde.csb |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
> On Thu, Sep 24, 2026 at 11:35:23AM +0200, Michael Banck wrote:
>
> But what could be done is add the pg_maintain (or something similar)
> default role as an allowed user for running this command besides
> Superuser.
+1, pg_maintain seems reasonable here.
Regarding the patch itself:
+typedef enum AlterSystemAction
+{
+ ALTER_SYSTEM_SET,
+ ALTER_SYSTEM_RESET,
+ ALTER_SYSTEM_RELOAD
+} AlterSystemAction;
What's the reason for separating SET and RESET once more? They're already
differentiated via setstmt->kind, aren't they?
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bharath Rupireddy | 2026-09-24 15:37:00 | Re: Teach pg_upgrade to deal with invalid databases |
| Previous Message | Heikki Linnakangas | 2026-09-24 15:32:57 | Re: [PATCH] Two remaining shmem attachment issues in single-user mode |