Improved psql tab completion for joins

Lists: pgsql-hackers
From: Andreas Karlsson <andreas(at)proxel(dot)se>
To: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Improved psql tab completion for joins
Date: 2024-11-16 16:59:18
Message-ID: [email protected]
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

Hi,

Here is a set of small patches which improve the tab completion of joins
in psql. The completion of select queries and DML is very primitive in
psql but since we already have completion of relation names after FROM
and JOIN I think these small additions fit and at least I find them
useful myself. I don't use USING that much in application code but when
writing ad hoc queries I use it quite a bit.

## 0001-Complete-LATERAL-keyword-for-joins.patch

Adds support for the LATERAL keyword after JOIN.

## 0002-Complete-ON-and-USING-keywords-for-joins.patch

Adds completion of the ON and USING keywords.

## 0002-Complete-ON-and-USING-keywords-for-joins.patch

Adds completion of the first USING column.

Andreas

Attachment Content-Type Size
0001-Complete-LATERAL-keyword-for-joins.patch text/x-patch 826 bytes
0002-Complete-ON-and-USING-keywords-for-joins.patch text/x-patch 1.4 KB
0003-Complete-first-member-of-USING-column-list.patch text/x-patch 963 bytes

From: Tomas Vondra <tomas(at)vondra(dot)me>
To: Andreas Karlsson <andreas(at)proxel(dot)se>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Improved psql tab completion for joins
Date: 2024-12-07 23:06:08
Message-ID: [email protected]
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

Hi,

On 11/16/24 17:59, Andreas Karlsson wrote:
> Hi,
>
> Here is a set of small patches which improve the tab completion of joins
> in psql. The completion of select queries and DML is very primitive in
> psql but since we already have completion of relation names after FROM
> and JOIN I think these small additions fit and at least I find them
> useful myself. I don't use USING that much in application code but when
> writing ad hoc queries I use it quite a bit.
>
> ## 0001-Complete-LATERAL-keyword-for-joins.patch
>
> Adds support for the LATERAL keyword after JOIN.
>
> ## 0002-Complete-ON-and-USING-keywords-for-joins.patch
>
> Adds completion of the ON and USING keywords.
>
> ## 0002-Complete-ON-and-USING-keywords-for-joins.patch
>
> Adds completion of the first USING column.
>

Thanks. On a cursory look these all seem reasonable to me. I'll do a bit
more review to make sure I didn't miss anything, and then I intend to
get this committed ...

regards

--
Tomas Vondra


From: Tomas Vondra <tomas(at)vondra(dot)me>
To: Andreas Karlsson <andreas(at)proxel(dot)se>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Improved psql tab completion for joins
Date: 2024-12-16 17:51:49
Message-ID: [email protected]
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

On 12/8/24 00:06, Tomas Vondra wrote:
> Hi,
>
> On 11/16/24 17:59, Andreas Karlsson wrote:
>> Hi,
>>
>> Here is a set of small patches which improve the tab completion of joins
>> in psql. The completion of select queries and DML is very primitive in
>> psql but since we already have completion of relation names after FROM
>> and JOIN I think these small additions fit and at least I find them
>> useful myself. I don't use USING that much in application code but when
>> writing ad hoc queries I use it quite a bit.
>>
>> ## 0001-Complete-LATERAL-keyword-for-joins.patch
>>
>> Adds support for the LATERAL keyword after JOIN.
>>
>> ## 0002-Complete-ON-and-USING-keywords-for-joins.patch
>>
>> Adds completion of the ON and USING keywords.
>>
>> ## 0002-Complete-ON-and-USING-keywords-for-joins.patch
>>
>> Adds completion of the first USING column.
>>
>
> Thanks. On a cursory look these all seem reasonable to me. I'll do a bit
> more review to make sure I didn't miss anything, and then I intend to
> get this committed ...
>

OK, pushed. Similarly to tho the other tab completion patches I
committed today, I chose not to squash the parts, even though these
changes seem to be in the same area. Seems tidier this way.

thanks!

--
Tomas Vondra


From: Andreas Karlsson <andreas(at)proxel(dot)se>
To: Tomas Vondra <tomas(at)vondra(dot)me>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Improved psql tab completion for joins
Date: 2024-12-16 18:35:18
Message-ID: [email protected]
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

On 12/16/24 6:51 PM, Tomas Vondra wrote:
> OK, pushed. Similarly to tho the other tab completion patches I
> committed today, I chose not to squash the parts, even though these
> changes seem to be in the same area. Seems tidier this way.

Thanks!

Personally I do not care either way. Most committers seem to prefer a
bit more squashed patches than I do but either is fine.

Andreas