Re: extend VacAttrStats to allow stavalues of different types

From: Jan Urbański <j(dot)urbanski(at)students(dot)mimuw(dot)edu(dot)pl>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Heikki Linnakangas <heikki(at)enterprisedb(dot)com>, Postgres - Patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: extend VacAttrStats to allow stavalues of different types
Date: 2008-06-02 15:58:48
Message-ID: 484418B8.6060004@students.mimuw.edu.pl
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

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.

--
Jan Urbanski
GPG key ID: E583D7D2

ouden estin

Attachment Content-Type Size
extend-vacattrstats-2.diff text/plain 3.5 KB

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Pavel Stehule 2008-06-03 11:03:06 SQL: table function support
Previous Message Tom Lane 2008-06-02 14:37:29 Re: extend VacAttrStats to allow stavalues of different types