Thanks for having a look. On Sat, 17 Aug 2024 at 23:21, Tels <nospam-pg-ab...@bloodgate.com> wrote: > Is it nec. to rotate existing_hash here before checking for isnull? > Because in case of isnull, isn't the result of the rotate thrown away?
Yeah, I think that it's worthwhile moving that to after the isnull check so as not to waste the effort. Unfortunately doing the same in the JIT code meant copying and pasting the code that emits the rotation code. The attached v5 patch includes this change. David
v5-0001-Speed-up-Hash-Join-by-making-ExprStates-support-h.patch
Description: Binary data
