lei yang <yanglei(dot)ovo(at)gmail(dot)com> writes:
> The tupleDesc parameter in the heap_getsysattr function is not used,
> and the TupleDesc structure does not contain system attributes.
> Maybe we can delete the parameter tupleDesc.
I do not think this is a great idea. We don't use the tupdesc today
perhaps, but it's plausible that it'd be needed in the future.
Different table AMs might have different sets of system columns.
Unless you can show a measurable performance gain from this change,
we should leave it alone.
regards, tom lane