| From: | Paul A Jungwirth <pj(at)illuminatedcomputing(dot)com> |
|---|---|
| To: | jian he <jian(dot)universality(at)gmail(dot)com> |
| Cc: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Subject: | Re: Move FOR PORTION OF checks out of analysis |
| Date: | 2026-06-11 21:37:45 |
| Message-ID: | CA+renyXyUz4nei=tTsX6Ohy1069FQnSFR063kV8LnOF1oUuKLQ@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Tue, Jun 9, 2026 at 1:11 AM jian he <jian(dot)universality(at)gmail(dot)com> wrote:
>
> missing errcode, we can change it to
>
> + if (contain_volatile_functions(parse->forPortionOf->targetRange))
> + ereport(ERROR,
> + errcode(ERRCODE_FEATURE_NOT_SUPPORTED),
> + errmsg("FOR PORTION OF bounds cannot contain
> volatile functions"));
Thanks for the note! Here is v9 with that change.
Yours,
--
Paul ~{:-)
pj(at)illuminatedcomputing(dot)com
| Attachment | Content-Type | Size |
|---|---|---|
| v9-0002-Move-FOR-PORTION-OF-volatile-check-into-planner.patch | text/x-patch | 2.0 KB |
| v9-0001-Reject-child-partition-FDWs-in-FOR-PORTION-OF.patch | text/x-patch | 10.4 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Jeff Davis | 2026-06-11 22:24:52 | Re: Commit Sequence Numbers and Visibility |
| Previous Message | Matheus Alcantara | 2026-06-11 21:31:07 | Re: LLVM JIT: any JIT-compiled query crashes (SIGILL) on a libLLVM 19 + ASAN build |