Re: [PATCH] support tab-completion for single quote input with equal sign

From: torikoshia <torikoshia(at)oss(dot)nttdata(dot)com>
To: tanghy(dot)fnst(at)fujitsu(dot)com
Cc: tgl(at)sss(dot)pgh(dot)pa(dot)us, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: [PATCH] support tab-completion for single quote input with equal sign
Date: 2023-01-10 13:01:01
Message-ID: 372b6e1165ca34eb738487b362fbf3b6@oss.nttdata.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thursday, July 22, 2021 1:05 PM, tanghy(dot)fnst(at)fujitsu(dot)com
<tanghy(dot)fnst(at)fujitsu(dot)com> wrote
> I found a problem when using tab-completion as follows:
>
> CREATE SUBSCRIPTION my_subscription
> CONNECTION 'host=localhost port=5432 dbname=postgres' [TAB]
>
> The word 'PUBLICATION' couldn't be auto completed as expected.

I too wondered about this behavior.

> v3-0001-support-tab-completion-for-CONNECTION-string-with.patch

I applied the patch and succeeded in the above case, but failed in the
below case.

=# CREATE SUBSCRIPTION s1 CONNECTION 'a=' PUBLICATION p1 <tab>

Before applying the patch, 'WITH (' was completed, but now it completes
nothing since it matches the below condition:

> 18 + else if ((HeadMatches("CREATE", "SUBSCRIPTION", MatchAny,
> "CONNECTION", MatchAny)))
> 19 + {

I updated the patch going along with the v3 direction.

What do you think?

--
Regards,

--
Atsushi Torikoshi
NTT DATA CORPORATION

Attachment Content-Type Size
v4-0001-support-tab-completion-for-CONNECTION-string-with.patch text/x-diff 2.4 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2023-01-10 13:12:38 Re: split TOAST support out of postgres.h
Previous Message Nazir Bilal Yavuz 2023-01-10 12:51:42 Re: Use windows VMs instead of windows containers on the CI