Re: extend VacAttrStats to allow stavalues of different types

From: "Heikki Linnakangas" <heikki(at)enterprisedb(dot)com>
To: Jan Urbański <j(dot)urbanski(at)students(dot)mimuw(dot)edu(dot)pl>
Cc: "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us>, "Postgres - Patches" <pgsql-patches(at)postgresql(dot)org>
Subject: Re: extend VacAttrStats to allow stavalues of different types
Date: 2008-07-01 10:33:24
Message-ID: 486A07F4.3070502@enterprisedb.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

Jan Urbański wrote:
> Tom Lane wrote:
>> 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.
>
> Right. Updated patch attached.

Thanks, committed.

--
Heikki Linnakangas
EnterpriseDB http://www.enterprisedb.com

In response to

Browse pgsql-patches by date

  From Date Subject
Next Message Heikki Linnakangas 2008-07-01 10:44:22 Re: [HACKERS] odd output in restore mode
Previous Message Simon Riggs 2008-07-01 08:39:08 Re: [HACKERS] odd output in restore mode