Re: Move FOR PORTION OF checks out of analysis

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

In response to

Browse pgsql-hackers by date

  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