Re: Remove unused for_all_tables field from AlterPublicationStmt

From: Álvaro Herrera <alvherre(at)kurilemu(dot)de>
To: Peter Smith <smithpb2250(at)gmail(dot)com>
Cc: Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>, Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Remove unused for_all_tables field from AlterPublicationStmt
Date: 2025-11-10 11:30:18
Message-ID: 202511101129.ajvvkwbbwgxs@alvherre.pgsql
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 2025-Nov-10, Peter Smith wrote:

> Therefore, it seems to me that the ALTER PUBLICATION syntax is
> incomplete. e.g. There also needs to be the ability to do ALTER
> PUBLICATION ADD/SET FOR ALL TABLES;
>
> Although the member that was removed might be currently unused, won't
> it be needed again whenever those missing ALTER PUBLICATION ADD/SET
> FOR ALL TABLES are implemented?

Sure, but it's easy to add again later when it is needed.

--
Álvaro Herrera PostgreSQL Developer — https://www.EnterpriseDB.com/
"I'm always right, but sometimes I'm more right than other times."
(Linus Torvalds)
https://lore.kernel.org/git/Pine(dot)LNX(dot)4(dot)58(dot)0504150753440(dot)7211(at)ppc970(dot)osdl(dot)org/

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Daniel Gustafsson 2025-11-10 11:41:01 Re: Move SLRU_PAGES_PER_SEGMENT to pg_config_manual.h
Previous Message Heikki Linnakangas 2025-11-10 11:29:08 Move SLRU_PAGES_PER_SEGMENT to pg_config_manual.h