Re: [PATCH] Statistics collection for CLUSTER command

From: Dimitri Fontaine <dimitri(at)2ndQuadrant(dot)fr>
To: Noah Misch <noah(at)leadboat(dot)com>
Cc: Vik Fearing <vik(dot)fearing(at)dalibo(dot)com>, Satoshi Nagayasu <snaga(at)uptime(dot)jp>, PG Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [PATCH] Statistics collection for CLUSTER command
Date: 2013-10-20 20:04:45
Message-ID: m2ppqzu2si.fsf@2ndQuadrant.fr
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Noah Misch <noah(at)leadboat(dot)com> writes:
> wrapper function. A backend change that would help here is to extend event
> triggers to cover the CLUSTER command, permitting you to inject monitoring
> after plain CLUSTER and dispense with the wrapper.

I didn't look in any level of details, but it might be as simple as
moving the T_ClusterStmt case from standard_ProcessUtility() down to the
Event Trigger friendly part known as ProcessUtilitySlow().

Regards,
--
Dimitri Fontaine
http://2ndQuadrant.fr PostgreSQL : Expertise, Formation et Support

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Noah Misch 2013-10-20 23:52:38 Re: proposal: lob conversion functionality
Previous Message David Fetter 2013-10-20 15:12:37 Re: Commitfest II CLosed