Hi Richard,
21.03.2023 14:52, Richard Guo wrote:
> Nice catch. This can also be seen on master.
>
> I searched the codes a little bit and found that in array_set_slice()
> and array_set_element() the 'newarray' is allocated with palloc0 and
> then the nulls bitmap is zeroed with
>
> MemSet(nullbitmap, 0, (nitems + 7) / 8);
>
> if havenulls is true. I wonder if we can do the same here.
I'm afraid that zeroing only bytes behind nitems bits is not enough, as outDatum() doesn't bother to calculate the exact
size of nulls bitmap, it just outputs all bytes of a datum (40 bytes in that case):
length = datumGetSize(value, typbyval, typlen);
...
for (i = 0; i < length; i++)
appendStringInfo(str, "%d ", (int) (s[i]));
Best regards,
Alexander