Re: Hybrid Hash/Nested Loop joins and caching results from subplans
David Rowley <dgrowleyml@gmail.com>
From: David Rowley <dgrowleyml@gmail.com>
To: Ibrar Ahmed <ibrar.ahmad@gmail.com>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, Andres Freund <andres@anarazel.de>, Konstantin Knizhnik <k.knizhnik@postgrespro.ru>,
Andy Fan <zhihui.fan1213@gmail.com>, Alvaro Herrera <alvherre@alvh.no-ip.org>, PostgreSQL Developers <pgsql-hackers@lists.postgresql.org>
Date: 2021-03-10T23:17:55Z
Lists: pgsql-hackers
Attachments
- v14-0001-Allow-estimate_num_groups-to-pass-back-further-d.patch (text/plain) patch v14-0001
- v14-0002-Allow-users-of-simplehash.h-to-perform-direct-de.patch (text/plain) patch v14-0002
- v14-0003-Add-Result-Cache-executor-node.patch (text/plain) patch v14-0003
- v14-0004-Remove-code-duplication-in-nodeResultCache.c.patch (text/plain) patch v14-0004
On Fri, 5 Mar 2021 at 00:16, Ibrar Ahmed <ibrar.ahmad@gmail.com> wrote: > This patch set no longer applies > http://cfbot.cputube.org/patch_32_2569.log > > Can we get a rebase? v14 should still apply. I think the problem is that the CFbot at best can only try and apply the latest .patch files that are on the thread in alphabetical order of the filename. The bot is likely just trying to apply the unrelated patch that was posted since I posted v14. I've attached the v14 version again. Hopefully, that'll make the CFbot happy. I'm also working on another version of the patch with slightly different planner code. I hope to reduce the additional planner overheads a bit with it. It should arrive here in the next day or two. David
Commits
-
Add Result Cache executor node (take 2)
- 9eacee2e62d8 14.0 landed
-
Add Result Cache executor node
- b6002a796dc0 14.0 landed
-
Allow estimate_num_groups() to pass back further details about the estimation
- ed934d4fa30f 14.0 landed
-
Allow users of simplehash.h to perform direct deletions
- ff53d7b159b9 14.0 landed
-
Cache if PathTarget and RestrictInfos contain volatile functions
- f58b230ed0db 14.0 landed
-
Fix pull_varnos' miscomputation of relids set for a PlaceHolderVar.
- 55dc86eca70b 14.0 cited