Re: Improve tab completion for COPY

From: Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com>
To: Nathan Bossart <nathandbossart(at)gmail(dot)com>, Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>
Cc: Yugo Nagata <nagata(at)sraoss(dot)co(dot)jp>, torikoshia <torikoshia(at)oss(dot)nttdata(dot)com>, Pgsql Hackers <pgsql-hackers(at)postgresql(dot)org>, rmt(at)lists(dot)postgresql(dot)org
Subject: Re: Improve tab completion for COPY
Date: 2025-06-10 23:56:48
Message-ID: 70372bdd-4399-4d5b-ab4f-6d4487a4911a@oss.nttdata.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 2025/06/11 5:33, Nathan Bossart wrote:
> On Tue, Jun 10, 2025 at 12:37:48PM -0700, Masahiko Sawada wrote:
>>>> (1) adds tab completion support for the REJECT_LIMIT option, which was
>>>> introduced in v18
>>>> (2) splits the tab completion logic between COPY FROM and COPY TO to
>>>> reflect their different options.
>>
>> [...]
>>
>> Given REJECT_LIMIT is a new v18 feature, it seems to me that (1) is an
>> oversight of this feature, but I also agree that it's not a bug and
>> doesn't block the release.
>>
>> How does the RMT feel about the change (1)? Nathan, would you be OK with that?
>
> 0001 sure seems like an oversight in commit 4ac2a9b to me. In any case, I
> argued that we should fix tab completion for unlogged partitioned tables in
> v18 [0], and this seems pretty similar.

BTW, I also have a similar patch [1] in my queue that I'm planning to commit.
Materialized views have been allowed in COPY TO since v18 (commit 534874fac0b),
and that patch improves tab completion to include them. For the same reason
discussed here, it might make sense to commit that patch in v18 as well.

Regards,

[1] https://postgr.es/m/CACJufxFxnSkikp+GormAGHcMTX1YH2HRXW1+3dJM9w7yY9hdsg@mail.gmail.com

--
Fujii Masao
NTT DATA Japan Corporation

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2025-06-10 23:56:54 Re: Safeguards against incorrect fd flags for fsync()
Previous Message Junwang Zhao 2025-06-10 23:41:54 Re: Use RELATION_IS_OTHER_TEMP where possible