| From: | Paul A Jungwirth <pj(at)illuminatedcomputing(dot)com> |
|---|---|
| To: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Cc: | jian he <jian(dot)universality(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Subject: | Move FOR PORTION OF checks out of analysis |
| Date: | 2026-05-15 21:16:57 |
| Message-ID: | CA+renyUte0_UJsJiDJQi82oaBsMJn=cct0Wn=vOqXtuDn=YYJA@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Starting a dedicated thread for this, based on [1]. To recap:
- FOR PORTION OF needs to reject FDWs in the planner/executor, not in
analysis, first to guarantee the status hasn't changed but also
because we need to check child partitions as well.
- We need to postpone checking for volatile functions too.
The first part is done by jian he's patch from that thread (with some
test revisions).
The second part is done by the second patch here.
There is no commitfest entry yet, so I'll make one from this thread.
[1] https://www.postgresql.org/message-id/626986.1776785090@sss.pgh.pa.us
Yours,
--
Paul ~{:-)
pj(at)illuminatedcomputing(dot)com
| Attachment | Content-Type | Size |
|---|---|---|
| v8-0001-Reject-child-partition-FDWs-in-FOR-PORTION-OF.patch | text/x-patch | 10.4 KB |
| v8-0002-Move-FOR-PORTION-OF-volatile-check-into-planner.patch | text/x-patch | 2.0 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Paul A Jungwirth | 2026-05-15 21:21:48 | Re: [PATCH] Fix null pointer dereference in PG19 |
| Previous Message | Paul A Jungwirth | 2026-05-15 21:04:21 | Re: FOR PORTION OF should reject GENERATED columns |