Re: Delay Memoize hashtable build until executor run

2024-01-29 Thread David Rowley
On Tue, 30 Jan 2024 at 12:25, David Rowley wrote: > I'll push this change to master only as there don't seem to have been > any complaints. We can reconsider that if someone complains. Pushed. David

Re: Delay Memoize hashtable build until executor run

2024-01-29 Thread David Rowley
On Fri, 26 Jan 2024 at 19:54, David Rowley wrote: > Currently, nodeMemoize.c builds the hashtable for the cache during > executor startup. This is not what is done in hash joins. I think we > should make the two behave the same way. I ran a few benchmarks on this, mostly for archive purposes. -

Delay Memoize hashtable build until executor run

2024-01-25 Thread David Rowley
Currently, nodeMemoize.c builds the hashtable for the cache during executor startup. This is not what is done in hash joins. I think we should make the two behave the same way. Per [1] and the corresponding discussion leading to that, making a possibly large allocation at executor startup can lea