Re: Add tab completion for SERVER and CONNECTION keywords in psql - Mailing list pgsql-hackers

From Yamaguchi Atsuo
Subject Re: Add tab completion for SERVER and CONNECTION keywords in psql
Date
Msg-id CAKSyus+UkjbWmoTHUUo6EKB+nzePQ9w8OQXUy92K6Kwp3cZMdQ@mail.gmail.com
Whole thread Raw
In response to Re: Add tab completion for SERVER and CONNECTION keywords in psql  (Masahiko Sawada <sawada.mshk@gmail.com>)
Responses Re: Add tab completion for SERVER and CONNECTION keywords in psql
List pgsql-hackers
Hi,

On Mon, Mar 23, 2026 at 1:14 PM Masahiko Sawada <sawada.mshk@gmail.com> wrote:
Hi,

On Sun, Mar 22, 2026 at 1:30 PM Yamaguchi Atsuo <acrobatcoder@gmail.com> wrote:
>
> Hi,
>
> This patch adds missing tab completion support in psql for:
>
> - SERVER keyword after ALTER/CREATE SUBSCRIPTION <name>
> - CONNECTION keyword after ALTER/CREATE FOREIGN DATA WRAPPER <name>
>
> Currently, typing "ALTER SUBSCRIPTION sub1 SERVER" and pressing Tab
> does not complete server names, even though the SERVER keyword itself
> is already suggested. Similarly, CONNECTION is not suggested as a
> completion candidate for FOREIGN DATA WRAPPER commands, despite being
> a valid option.
>
> The patch is generated against the current master branch.

It's a fix for an oversight of commit 8185bb5347 rather than proposing
a new tab-completion. Good catch. Regarding the patch, I have one
comment:

    /* ALTER FOREIGN DATA WRAPPER <name> */
    else if (Matches("ALTER", "FOREIGN", "DATA", "WRAPPER", MatchAny))
        COMPLETE_WITH("HANDLER", "VALIDATOR", "NO",
-                     "OPTIONS", "OWNER TO", "RENAME TO");
+                     "OPTIONS", "OWNER TO", "RENAME TO", "CONNECTION");

I think it's better to maintain the keywords in the list in alphabetical order.

Regards,

--
Masahiko Sawada
Amazon Web Services: https://aws.amazon.com

 Thank you for the review. I have updated the code to maintain the keywords in alphabetical order.

Regards,
Atsuo Yamaguchi
Attachment

pgsql-hackers by date:

Previous
From: Nathan Bossart
Date:
Subject: Re: Fixes inconsistent behavior in vacuum when it processes multiple relations
Next
From: Srinath Reddy Sadipiralla
Date:
Subject: Re: Adding REPACK [concurrently]