From: | Amit Langote <amitlangote09(at)gmail(dot)com> |
---|---|
To: | Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com> |
Cc: | David Rowley <dgrowleyml(at)gmail(dot)com>, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, Dmitry Dolgov <9erthalion6(at)gmail(dot)com>, tomas(at)vondra(dot)me, vignesh C <vignesh21(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Richard Guo <guofenglinux(at)gmail(dot)com> |
Subject: | Re: Reducing memory consumed by RestrictInfo list translations in partitionwise join planning |
Date: | 2025-04-04 08:52:08 |
Message-ID: | CA+HiwqECgJnrvj=ey822QOQvePSZKDRrtT6v-9jOrhetqyKAqg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Fri, Apr 4, 2025 at 5:48 PM Ashutosh Bapat
<ashutosh(dot)bapat(dot)oss(at)gmail(dot)com> wrote:
> On Fri, Apr 4, 2025 at 2:04 PM Amit Langote <amitlangote09(at)gmail(dot)com> wrote:
> > I’ve now marked this as committed after pushing the patches earlier today.
>
> Thanks a lot.
Thank you too for working on it.
> > I realize the CF entry was originally about the project to reduce
> > memory usage during partitionwise join planning, but we ended up
> > committing something else. I suppose we can create a new entry if and
> > when we pick that original work back up.
>
> Will create a new CF entry just to keep the patch floated.
Sounds good.
Saving memory where we can does seem worthwhile, as long as the
approach stays simple. If there’s any doubt on that front, maybe it’s
worth spending a bit more time to see if things can be simplified.
--
Thanks, Amit Langote
From | Date | Subject | |
---|---|---|---|
Next Message | Mahendra Singh Thalor | 2025-04-04 09:12:32 | Re: Non-text mode for pg_dumpall |
Previous Message | Ashutosh Bapat | 2025-04-04 08:47:50 | Re: Reducing memory consumed by RestrictInfo list translations in partitionwise join planning |