> Hello KaiGai-san,
>
> I've discovered a bug in the proposed patch - when resetting the hash
> table after the first batch, it does this:
>
> memset(hashtable->buckets, 0, sizeof(nbuckets * sizeof(HashJoinTuple)));
>
> The first 'sizeof' is bogus, so this only zeroes the first 8 bytes of
> the array (usually resulting in crashes due to invalid pointers).
>
> I fixed it to
>
> memset(hashtable->buckets, 0, nbuckets * sizeof(HashJoinTuple));
>
> Fixed patch attached (marked as v2).
>
Thanks, it was my bug, but oversight.
I want committer to push this fix.
--
NEC Business Creation Division / PG-Strom Project
KaiGai Kohei <kaigai@ak.jp.nec.com>
