Skip site navigation (1) Skip section navigation (2)

Re: 9.2 Reviewfest - logging hooks patch

From: Martin Pihlak <martin(dot)pihlak(at)gmail(dot)com>
To: Christopher Maujean <cmaujean(at)pincsolutions(dot)com>
Cc: "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: 9.2 Reviewfest - logging hooks patch
Date: 2012-01-15 17:36:53
Message-ID: (view raw, whole thread or download thread mbox)
Lists: pgsql-hackers
On 01/13/2012 05:29 AM, Christopher Maujean wrote:
> Patch
> <>
> In reviewing the referenced patch, we found that it has no developer
> documentation and no regression tests. While it compiles and installs,
> there is nothing to tell us how to use (or test) it.

Thanks for reviewing!

I think the regression tests should be part of the module that uses
the hooks. Alas, there are none that we'd want to ship with Postgres
at this time.

The best I could offer for testing the hooks is a log forwarding module
I've been working on. Basically, what it does is that it installs the
hooks, looks at the message emitted by backend and if needed sends it
away via UDP. Available at:

To install, run "make install" and modify postgresql.conf to include:

shared_preload_libraries = 'pg_logforward'
logforward.target_names = 'jsonsrv'
logforward.jsonsrv_host = ''
logforward.jsonsrv_port = 23456

Now, after restarting postgres, you could use "nc 23456 -u -l" to
receive the log. Alternatively use testing/ as a
log "server".

About documentation - it sure would be nice to have all the hooks
written up somewhere. Any suggestions as to where to put it?


In response to

pgsql-hackers by date

Next:From: Joachim WielandDate: 2012-01-15 18:01:43
Subject: patch for parallel pg_dump
Previous:From: Tom LaneDate: 2012-01-15 17:20:44
Subject: Re: pgstat documentation tables

Privacy Policy | About PostgreSQL
Copyright © 1996-2017 The PostgreSQL Global Development Group