Re: Patch for pgstatindex

Lists: pgadmin-hackers
From: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
To: pgadmin-hackers(at)postgresql(dot)org, Dave Page <dpage(at)postgresql(dot)org>
Subject: Patch for pgstatindex
Date: 2007-02-28 23:46:22
Message-ID: [email protected]
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgadmin-hackers

Hi,

Finally, I found some time to work on this patch. It adds support for
pgstatindex function (8.2 pgstattuple contrib module).

Please, review it.

Thanks.

--
Guillaume.
<!-- http://abs.traduc.org/
http://lfs.traduc.org/
http://docs.postgresqlfr.org/ -->

Attachment Content-Type Size
pgstatindex.patch text/x-patch 7.5 KB

From: Dave Page <dpage(at)postgresql(dot)org>
To: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
Cc: pgadmin-hackers(at)postgresql(dot)org
Subject: Re: Patch for pgstatindex
Date: 2007-03-01 10:12:59
Message-ID: [email protected]
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgadmin-hackers

Guillaume Lelarge wrote:
> Hi,
>
> Finally, I found some time to work on this patch. It adds support for
> pgstatindex function (8.2 pgstattuple contrib module).
>
> Please, review it.

Looks OK from a read of the patch - I assume it's basically the same as
the pgstattuple patch as it ended up when committed?

Regard,s Dave


From: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
To: Dave Page <dpage(at)postgresql(dot)org>
Cc: pgadmin-hackers(at)postgresql(dot)org
Subject: Re: Patch for pgstatindex
Date: 2007-03-01 17:15:08
Message-ID: [email protected]
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgadmin-hackers

Dave Page a ecrit le 01/03/2007 11:12:
> Guillaume Lelarge wrote:
>> Finally, I found some time to work on this patch. It adds support for
>> pgstatindex function (8.2 pgstattuple contrib module).
>>
>> Please, review it.
>
> Looks OK from a read of the patch - I assume it's basically the same as
> the pgstattuple patch as it ended up when committed?
>

Basically, yes. I did it by sort of duplicating the pgstattuple code.

--
Guillaume.


From: Dave Page <dpage(at)postgresql(dot)org>
To: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
Cc: pgadmin-hackers(at)postgresql(dot)org
Subject: Re: Patch for pgstatindex
Date: 2007-03-02 11:00:34
Message-ID: [email protected]
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgadmin-hackers

Guillaume Lelarge wrote:
> Hi,
>
> Finally, I found some time to work on this patch. It adds support for
> pgstatindex function (8.2 pgstattuple contrib module).

Thanks Guillaume.

I've applied your patch with a minor change to enable the feature for
primary key and unique constraints as well.

Regards, Dave