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
