Re: Improve tab completion for COPY

From: Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>
To: Yugo Nagata <nagata(at)sraoss(dot)co(dot)jp>, Nathan Bossart <nathandbossart(at)gmail(dot)com>
Cc: torikoshia <torikoshia(at)oss(dot)nttdata(dot)com>, Pgsql Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Improve tab completion for COPY
Date: 2025-06-10 19:37:48
Message-ID: CAD21AoBXB+bWUJkBnBpPme6j6HgdTtgszi7naho7OCbasEz4Yg@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, Jun 9, 2025 at 7:59 PM Yugo Nagata <nagata(at)sraoss(dot)co(dot)jp> wrote:
>
> On Tue, 10 Jun 2025 11:15:09 +0900
> torikoshia <torikoshia(at)oss(dot)nttdata(dot)com> wrote:
>
> > On 2025-06-03 17:58, Yugo Nagata wrote:
> > > Thank you for updating the patch.
> > > It looks good and I confirmed that this works as expected.
> >
> > Thanks for your review!
> >
> > BTW this is a small patch, but it does two things:
> >
> > (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.
> >
> > While maybe (2) should be postponed to v19 or later, I think it would be
> > better to include (1) in v18.
> > So I’ve split them into separate patches accordingly.
>
> I am not convinced whether (1) should be regarded as a v18-related oversight
> or a new feature, but if this is a oversight, this should be added to the open
> items list?

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?

Regards,

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

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Bruce Momjian 2025-06-10 19:38:11 Re: Proposal: Global Index for PostgreSQL
Previous Message Sami Imseih 2025-06-10 19:05:16 Re: add function for creating/attaching hash table in DSM registry