amul sul <sul_amul(at)yahoo(dot)co(dot)in> writes:
> For more granularity,I think passingForm_pg_attribute toexamine_attribute() function rather than passing Relation will be more relevant & makes it simple to understand.
I don't find that to be a good idea at all. It makes examine_attribute
inconsistent with most other functions in analyze.c, and it limits our
ability to add logic inside that function that might want to look at
other properties of the relation.
Even without that argument, moving the responsibility for initializing
stats->tupattnum to the callers of examine_attribute is certainly a
net loss in readability and reliability.
regards, tom lane