| From: | Richard Guo <guofenglinux(at)gmail(dot)com> |
|---|---|
| To: | Amit Langote <amitlangote09(at)gmail(dot)com> |
| Cc: | Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com>, Andrei Lepikhov <a(dot)lepikhov(at)postgrespro(dot)ru>, Tomas Vondra <tomas(dot)vondra(at)enterprisedb(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, vignesh C <vignesh21(at)gmail(dot)com> |
| Subject: | Re: Memory consumed by child SpecialJoinInfo in partitionwise join planning |
| Date: | 2024-03-25 10:25:19 |
| Message-ID: | CAMbWs4-_+ng9LXinDOqMHnq7On=fVL5oXV+4kd-B+J4DyUwJ_w@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Mon, Mar 25, 2024 at 5:17 PM Amit Langote <amitlangote09(at)gmail(dot)com>
wrote:
> I've pushed this now.
>
> When updating the commit message, I realized that you had made the
> right call to divide the the changes around not translating the dummy
> SpecialJoinInfos into a separate patch. So, I pushed it like that.
Sorry I'm late for the party. I noticed that in commit 6190d828cd the
comment of init_dummy_sjinfo() mentions the non-existing 'rel1' and
'rel2', which should be addressed. Also, I think we can make function
consider_new_or_clause() call init_dummy_sjinfo() to simplify the code a
bit.
Attached is a trivial patch for the fixes.
Thanks
Richard
| Attachment | Content-Type | Size |
|---|---|---|
| v1-0001-Fix-the-comment-of-init_dummy_sjinfo.patch | application/octet-stream | 2.8 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Alexander Korotkov | 2024-03-25 10:28:36 | Re: Add SPLIT PARTITION/MERGE PARTITIONS commands |
| Previous Message | Ashutosh Bapat | 2024-03-25 10:10:00 | Re: Memory consumed by child SpecialJoinInfo in partitionwise join planning |