| From: | "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com> |
|---|---|
| To: | Peter Smith <smithpb2250(at)gmail(dot)com> |
| Cc: | Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com>, Shlok Kyal <shlok(dot)kyal(dot)oss(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: Improve documentation of publication privilege checks |
| Date: | 2025-12-24 00:52:37 |
| Message-ID: | CAKFQuwYO=F9Odm2j86BmSYVuyR_tyR899Oj+WVuGfhU=N_LCkw@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Tue, Dec 23, 2025 at 4:43 PM Peter Smith <smithpb2250(at)gmail(dot)com> wrote:
> Hi Shlok/Chao-San.
>
> How about this alternative wording:
>
> <para>
> To create a publication using any of <literal>FOR ALL TABLES</literal>,
> <literal>FOR ALL SEQUENCES</literal>, or
> <literal>FOR TABLES IN SCHEMA</literal>, the user must be a superuser.
> To alter a publication using <literal>ADD TABLE</literal>, the user must
> have ownership rights on the table. To alter a publication using
> <literal>ADD TABLES IN SCHEMA</literal>, the user must be a superuser.
> </para>
>
>
I initially preferred Chao Li's version but upon deeper consideration I've
settled on this variant. The conjunctions in the other are nice, but I've
come to like how create and alter are better separated here. And the
choice to list "add table" first breaks up the string of superuser required
commands when switching from creating to altering.
Kinda feel we should start this with the individual table creation case
though:
To create a publication using FOR TABLE, the user must have ownership
rights on all listed tables. To create a publication using any of ... the
user must be a superuser. To alter ...
The alter case likewise accepts multiple tables...
David J.
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Chao Li | 2025-12-24 01:28:37 | Refactor replication origin state reset helpers |
| Previous Message | Dharin Shah | 2025-12-24 00:47:16 | Re: Fwd: [PATCH] Add zstd compression for TOAST using extended header format |