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