Hi~
Sorry for multiple comments in separate mail. Just found that the initialization
seems redundant since we have used palloc0?
> + istate = (HeapInsertState *) palloc0(sizeof(HeapInsertState));
> + istate->bistate = NULL;
> + istate->mistate = NULL;
---
Regards, Jingtang