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
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 |