Re: Reducing memory consumed by RestrictInfo list translations in partitionwise join planning
Ashutosh Bapat <ashutosh.bapat.oss@gmail.com>
From: Ashutosh Bapat <ashutosh.bapat.oss@gmail.com>
To: Alvaro Herrera <alvherre@alvh.no-ip.org>
Cc: Dmitry Dolgov <9erthalion6@gmail.com>, tomas@vondra.me,
vignesh C <vignesh21@gmail.com>, pgsql-hackers <pgsql-hackers@postgresql.org>,
Richard Guo <guofenglinux@gmail.com>, David Rowley <dgrowleyml@gmail.com>
Date: 2025-02-20T11:58:48Z
Lists: pgsql-hackers
Commits
Same data as JSON:
GET /api/v1/messages/:b64id/commits
the thread's linked commits as JSON, with link sources.
API reference →
-
Add assertion to verify derived clause has constant RHS
- 887160d1beae 18.0 landed
-
Make derived clause lookup in EquivalenceClass more efficient
- 88f55bc97622 18.0 landed
-
Doc: improve documentation for jsonpath behavior.
- 7014c9a4bba2 17.0 cited
-
Work around implementation restriction in adjust_appendrel_attrs.
- 767c598954bb 16.0 cited
Attachments
- 0001-RestrictInfo-hash-table-interface-20250220.patch (text/x-patch) patch 0001
- 0004-Avoid-translating-RestrictInfo-repeatedly-20250220.patch (text/x-patch) patch 0004
- 0003-Use-RestrictInfo-hash-table-for-storing-EC--20250220.patch (text/x-patch) patch 0003
- 0002-Compact-PlannerInfo-to-restore-its-previous-20250220.patch (text/x-patch) patch 0002
On Tue, Feb 4, 2025 at 4:07 PM Ashutosh Bapat <ashutosh.bapat.oss@gmail.com> wrote: > > If we are not interested in saving memory, there is a simpler way to > improve planning time by adding a hash table per equivalence class to > store the derived clauses, instead of a linked list, when the number > of derived clauses is higher than a threshold (say 32 same as the > threshold for join_rel_list. Maybe that approach will yield stable > planning time. > PFA patchset with conflict, with latest HEAD, resolved. I have dropped 0005 from the previous patchset since it didn't show any significant performance difference. Other than these two things, the patchset is same as the previous one. -- Best Wishes, Ashutosh Bapat