Re: psql: add tab completion for subscription wal_receiver_timeout

From: Xuneng Zhou <xunengzhou(at)gmail(dot)com>
To: Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Fujii Masao <masao(dot)fujii(at)gmail(dot)com>
Subject: Re: psql: add tab completion for subscription wal_receiver_timeout
Date: 2026-06-24 07:52:04
Message-ID: CABPTF7ULONod2wunZA9htB0AiA6ExSZX7YKW0hWjAurBOC5vuA@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi Evan,

On Wed, Jun 24, 2026 at 11:50 AM Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com> wrote:
>
> Hi,
>
> While testing "[fb80f388f] Add per-subscription wal_receiver_timeout setting", I noticed that wal_receiver_timeout is missing from tab completion for both CREATE SUBSCRIPTION WITH and ALTER SUBSCRIPTION SET:
> ```
> evantest=# create subscription ss with (
> BINARY CREATE_SLOT FAILOVER PASSWORD_REQUIRED SLOT_NAME TWO_PHASE
> CONNECT DISABLE_ON_ERROR MAX_RETENTION_DURATION RETAIN_DEAD_TUPLES STREAMING
> COPY_DATA ENABLED ORIGIN RUN_AS_OWNER SYNCHRONOUS_COMMIT
> evantest=# alter subscription ss set (
> BINARY FAILOVER ORIGIN RETAIN_DEAD_TUPLES SLOT_NAME SYNCHRONOUS_COMMIT
> DISABLE_ON_ERROR MAX_RETENTION_DURATION PASSWORD_REQUIRED RUN_AS_OWNER STREAMING TWO_PHASE
> ```
>
> This is a tiny patch to fix that.
>

The patch LGTM. I've verified that SUBOPT_WAL_RECEIVER_TIMEOUT is in
both the CREATE and ALTER supported-option masks.

--
Regards,
Xuneng Zhou
HighGo Software Co., Ltd.

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Akshay Joshi 2026-06-24 08:23:13 Re: [PATCH] Add pg_get_table_ddl() to reconstruct CREATE TABLE statements
Previous Message Chao Li 2026-06-24 07:47:36 Re: bytea(uuid) missing proleakproof?