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

Re: Review: Patch for contains/overlap of polygons

From: Joshua Tolley <eggyknap(at)gmail(dot)com>
To: Robert Haas <robertmhaas(at)gmail(dot)com>
Cc: Bruce Momjian <bruce(at)momjian(dot)us>, Josh Williams <joshwilliams(at)ij(dot)net>,Teodor Sigaev <teodor(at)sigaev(dot)ru>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: Review: Patch for contains/overlap of polygons
Date: 2009-08-09 19:27:30
Message-ID: 20090809192730.GC24450@eddie (view raw or flat)
Thread:
Lists: pgsql-hackers
On Sun, Aug 09, 2009 at 02:29:44PM -0400, Robert Haas wrote:
> On Sun, Aug 9, 2009 at 2:20 PM, Bruce Momjian<bruce(at)momjian(dot)us> wrote:
> > This is a nice section layout for a patch review report --- should we
> > provide an email template like this one for reviewers to use?
> 
> We could, but it might be over-engineering.  Those particular section
> headers might not be applicable to someone else's review.

I've just added a link to this email to the "Reviewing a Patch" wiki page
(http://wiki.postgresql.org/wiki/Reviewing_a_Patch). Do with it as you see fit
:)

--
Joshua Tolley / eggyknap
End Point Corporation
http://www.endpoint.com

In response to

Responses

pgsql-hackers by date

Next:From: Tom LaneDate: 2009-08-09 19:57:00
Subject: Re: machine-readable explain output v4
Previous:From: Tom LaneDate: 2009-08-09 19:00:32
Subject: Re: mixed, named notation support

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