Re: pgstattuple extension for indexes

From: Alvaro Herrera <alvherre(at)commandprompt(dot)com>
To: Satoshi Nagayasu <nagayasus(at)nttdata(dot)co(dot)jp>
Cc: Bruce Momjian <bruce(at)momjian(dot)us>, pgsql-patches(at)postgresql(dot)org, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, ITAGAKI Takahiro <itagaki(dot)takahiro(at)oss(dot)ntt(dot)co(dot)jp>, ishii(at)sraoss(dot)co(dot)jp
Subject: Re: pgstattuple extension for indexes
Date: 2006-08-14 04:51:40
Message-ID: 20060814045140.GA477@alvh.no-ip.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Satoshi Nagayasu wrote:
> Bruce,
>
> Attached patch has been cleaned up,
> and modified to be able to work with CVS HEAD.

I was thinking, isn't it a lot cleaner to define the functions to use
OUT parameters instead of having to define a custom type for each?

Also, in 8.2 there is a uninstall SQL script -- ISTM you should put the
DROP commands there, not in the install script.

--
Alvaro Herrera http://www.CommandPrompt.com/
The PostgreSQL Company - Command Prompt, Inc.

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Martijn van Oosterhout 2006-08-14 07:41:50 Re: Google SoC--Idea Request
Previous Message Tom Lane 2006-08-14 03:28:20 Re: [PATCHES] Adding fulldisjunctions to the contrib

Browse pgsql-patches by date

  From Date Subject
Next Message Tom Lane 2006-08-14 11:58:12 Re: pgstattuple extension for indexes
Previous Message Tom Lane 2006-08-14 03:28:20 Re: [PATCHES] Adding fulldisjunctions to the contrib