On Mon, May 24, 2021 at 02:46:11PM +0530, Dilip Kumar wrote:
> Like this.
> if (TupleDescAttr(newTupDesc, i)->attisdropped)
> isnull[i] = true;
> else
> tup_values[i] = values[i];
That would work. Your suggestion, as I understood it first, makes the
code simpler by not using tup_values at all as the set of values[] is
filled when the values and nulls are extracted. So I have gone with
this simplification, and applied the patch (moved a bit the comments
while on it).
--
Michael