Re: Patch review for logging hooks (CF 2012-01)

From: Greg Smith <greg(at)2ndQuadrant(dot)com>
To: pgsql-hackers(at)postgresql(dot)org
Subject: Re: Patch review for logging hooks (CF 2012-01)
Date: 2012-01-20 07:47:15
Message-ID: 4F191C03.4000107@2ndQuadrant.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 01/18/2012 04:23 PM, Marti Raudsepp wrote:
> The updated patch looks good, marking as 'Ready for Committer'

Patches without documentation are never ready for commit. For this one,
I'm not sure if that should be in the form of a reference example in
contrib, or just something that documents that the hook exists and what
the ground rules are for grabbing it.

--
Greg Smith 2ndQuadrant US greg(at)2ndQuadrant(dot)com Baltimore, MD
PostgreSQL Training, Services, and 24x7 Support www.2ndQuadrant.com

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Fujii Masao 2012-01-20 09:48:59 Re: Online base backup from the hot-standby
Previous Message Nikhil Sontakke 2012-01-20 06:02:25 Re: Review of patch renaming constraints