Re: [HACKERS] get_relation_stats_hook()

From: Simon Riggs <simon(at)2ndQuadrant(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Gregory Stark <stark(at)enterprisedb(dot)com>
Cc: Alvaro Herrera <alvherre(at)commandprompt(dot)com>, List pgsql-patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: [HACKERS] get_relation_stats_hook()
Date: 2008-09-26 14:52:20
Message-ID: 1222440740.4445.966.camel@ebony.2ndQuadrant
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches


On Wed, 2008-08-06 at 23:38 +0100, Simon Riggs wrote:
> On Wed, 2008-08-06 at 16:37 +0100, Simon Riggs wrote:
>
> > I'll submit the fully working plugin once we've stabilised the API. It's
> > designed as a contrib module, so it can go in pgfoundry or contrib.
>
> OK, here's fully working plugin, plus API patch.
>
> I expect to open a pgfoundry project for the plugin, but will wait for
> the main patch review.
>
> I've tried the APIs three different ways and this seems cleanest and
> most widely applicable approach. It's possible to add calls in more
> places, but I haven't done this for reasons discussed previously.

New version of Postgres patch, v5. Implements suggested changes.
Ready for review and apply.

New version of stats plugin, v3. Works with v5.
Corrected problems:
* now loads using preload_shared_libraries as well as LOAD
* example test script fix

--
Simon Riggs www.2ndQuadrant.com
PostgreSQL Training, Services and Support

Attachment Content-Type Size
stat_hooks.v5.patch text/x-patch 9.2 KB
TOM.v3.tar application/x-tar 30.0 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Stefan Kaltenbrunner 2008-09-26 14:59:44 planned maintenance downtime - tribble.postgresql.org
Previous Message KaiGai Kohei 2008-09-26 14:07:06 Re: Updates of SE-PostgreSQL 8.4devel patches

Browse pgsql-patches by date

  From Date Subject
Next Message Tom Lane 2008-09-28 18:02:25 Re: [PATCHES] Infrastructure changes for recovery
Previous Message Simon Riggs 2008-09-26 11:42:06 Re: [PATCHES] Infrastructure changes for recovery