From: | Atsushi Torikoshi <torikoshia(dot)tech(at)gmail(dot)com> |
---|---|
To: | Jim Jones <jim(dot)jones(at)uni-muenster(dot)de> |
Cc: | torikoshia <torikoshia(at)oss(dot)nttdata(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org |
Subject: | Re: COPY TO: provide hint when WHERE clause is used |
Date: | 2025-08-31 22:58:21 |
Message-ID: | CAM6-o=DVOd-jpO6WparhuPjOG5QkNiyspdPzeOrDT6x_JAqDMw@mail.gmail.com |
Views: | Whole Thread | Raw Message | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Sun, Aug 31, 2025 at 3:14 AM Jim Jones <jim(dot)jones(at)uni-muenster(dot)de> wrote:
>
> Hi
>
> On 19.08.25 03:35, torikoshia wrote:
> > Considering what the user was trying to do, it might be helpful to
> > provide a similar hint in this case as well.
> > I’ve attached a patch that adds such a hint.
> >
> > What do you think?
> >
>
> +1
>
> I tested the patch and the error message now has the intended HINT ...
>
> postgres=# COPY t1 TO STDOUT WHERE i > 5;
> ERROR: WHERE clause not allowed with COPY TO
> LINE 1: COPY t1 TO STDOUT WHERE i > 5;
> ^
> HINT: Try the COPY (SELECT ... WHERE ...) TO variant.
>
> ... which aligns with the HINT from COPY TO from views
>
> postgres=# COPY v1 TO STDOUT;
> ERROR: cannot copy from view "v1"
> HINT: Try the COPY (SELECT ...) TO variant.
Thanks for your review!
Updated the patch to pass regression tests.
Attachment | Content-Type | Size |
---|---|---|
v2-0001-COPY-TO-provide-hint-when-WHERE-clause-is-used.patch | application/x-patch | 1.5 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Lukas Fittl | 2025-08-31 23:57:01 | Stack-based tracking of per-node WAL/buffer usage |
Previous Message | Jelte Fennema-Nio | 2025-08-31 20:53:23 | Re: PG 18 release notes draft committed |