Re: extend VacAttrStats to allow stavalues of different types

"Heikki Linnakangas" <heikki@enterprisedb.com> writes:
> Jan Urbański wrote:
>> +      * XXX or maybe fall back on attrtype-> stuff when these are NULL? That way
>> +      * we won't break other people's custom typanalyze functions. Not sure if
>> +      * any exist, though.

> I tried to google for a user defined data type with a custom typanalyze
> function but didn't find anything, so I don't think it's an issue.

Oh, it absolutely is an issue.  PostGIS has them, if no one else does.

I think the correct solution is to initialize the fields to match the
column type before calling the typanalyze function.  Then you don't
break compatibility for existing typanalyze functions.  It's also less
code, since the standard typanalyze functions can rely on those preset
values.

            regards, tom lane