| From: | Itagaki Takahiro <itagaki(dot)takahiro(at)oss(dot)ntt(dot)co(dot)jp> |
|---|---|
| To: | pgsql-hackers(at)postgresql(dot)org |
| Cc: | Euler Taveira de Oliveira <euler(at)timbira(dot)com> |
| Subject: | Re: ProcessUtility_hook |
| Date: | 2009-10-21 01:14:49 |
| Message-ID: | 20091021100901.955B.52131E4D@oss.ntt.co.jp |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Here is a patch to add ProcessUtility_hook to handle all DDL
commands in user modules. (ProcessUtility_hook_20091021.patch)
It adds a global variable 'ProcessUtility_hook' and
export the original function as 'standard_ProcessUtility'.
Euler Taveira de Oliveira <euler(at)timbira(dot)com> wrote:
> > pg_stat_statements module also handle DDL commands as same as normal
> > queries. Fortunately, autovacuum calls call vacuum() directly,
> > so the statistics will not filled with VACUUM and ANALYZE commands
> > by autovacuum.
> >
> I don't look at the code but maybe there is a way to hook something there too.
> But I'd leave it alone for now, unless it's few lines of code.
I see it is debatable whether pg_stat_statements should support the hook.
So I split the change in another patch. (pgss_utility_hook_20091021.patch)
Regards,
---
ITAGAKI Takahiro
NTT Open Source Software Center
| Attachment | Content-Type | Size |
|---|---|---|
| ProcessUtility_hook_20091021.patch | application/octet-stream | 2.5 KB |
| pgss_utility_hook_20091021.patch | application/octet-stream | 5.6 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | u235sentinel | 2009-10-21 03:07:05 | Re: postgres 8.3.8 and Solaris 10_x86 64 bit problems? |
| Previous Message | Itagaki Takahiro | 2009-10-21 01:04:10 | Re: Going, going, GUCs! |