| From: | Tender Wang <tndrwang(at)gmail(dot)com> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com>, Duncan Sands <duncan(dot)sands(at)deepbluecap(dot)com>, pgsql-bugs(at)lists(dot)postgresql(dot)org |
| Subject: | Re: Attribute of type record has wrong type error with MERGE ... WHEN NOT MATCHED BY SOURCE THEN DELETE |
| Date: | 2025-03-11 06:11:47 |
| Message-ID: | CAHewXNn0fA6Mqu9fV+-oy6V642sxxHWBtbZ+KJuvFMEboYqjAw@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-bugs |
Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> 于2025年3月11日周二 03:11写道：
> I wrote:
> > Yeah, I think we can likely get away with that. We cannot back-patch
> > the changes that added relid to the outfuncs/readfuncs representation,
> > which means that the RTE's relid won't propagate to parallel workers,
> > but I don't see why they'd need it. We only need that info to get
> > as far as planning. I've not tried though.
>
> OK, the attached patch for v15 passes check-world, with or without
> force_parallel_mode. I'm inclined to commit the rewriteHandler.c
> and parsenodes.h bits in a separate patch for commit log visibility.
>
The attached patch looks better than my proposed code.
LGTM.
--
Thanks,
Tender Wang
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bruno Friedmann | 2025-03-11 08:23:10 | Re: BUG #18833: libpq.so doesn't contain declared symbol in rpm --provides |
| Previous Message | Devrim Gündüz | 2025-03-11 02:21:13 | Re: BUG #18833: libpq.so doesn't contain declared symbol in rpm --provides |