From: Greg Smith <greg@2ndQuadrant.com>
To: Peter Eisentraut <peter_e@gmx.net>
Cc: pgsql-hackers@postgresql.org
Subject: Re: automating CF submissions (was xlog location arithmetic)
Date: Tue, 17 Jan 2012 13:19:13 -0500
Message-ID: <4F15BBA1.5010701@2ndQuadrant.com> (raw)
In-Reply-To: <1326819026.2820.1.camel@vanquo.pezone.net>
References: <4EDD97E5.8080201@timbira.com>
<CAHGQGwFBF5D22cpw5gqXqLUDJLoJOfi_rc7BDg0==ekX4887tg@mail.gmail.com>
<4F118D45.3060801@timbira.com>
<4F122967.6070400@2ndQuadrant.com>
<CABwTF4WpYyMy2Uk2j=oTB8RB-cf2EmRJ=wfiW2r3Gj5SBb+fAA@mail.gmail.com>
<4F125998.5020204@2ndQuadrant.com>
<4F148D29.7080504@agliodbs.com>
<4F14A3EE.8000402@2ndQuadrant.com>
<1326819026.2820.1.camel@vanquo.pezone.net>
On 01/17/2012 11:50 AM, Peter Eisentraut wrote:
> On mån, 2012-01-16 at 17:25 -0500, Greg Smith wrote:
>> The most reasonable answer to this is for people to publish a git repo
>> URL in addition to the "official" submission of changes to the list in
>> patch form.
> Note that the original complaint was that for the occasional reviewer,
> the current system takes at least 5 partially redundant steps in two
> different systems. I doubt that adding a third system and more
> partially redundant steps it going to help that.
Publishing the submission via git is an extra step for the patch
submitter. If that happens, the reviewer can test just be cloning that,
instead of first closing the PostgreSQL one then applying the patch. It
removes the "how do I fish the patch out of the archives?" problem from
the reviewer's side of things.
--
Greg Smith 2ndQuadrant US greg@2ndQuadrant.com Baltimore, MD
PostgreSQL Training, Services, and 24x7 Support www.2ndQuadrant.com
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-hackers@postgresql.org
Cc: greg@2ndQuadrant.com, peter_e@gmx.net
Subject: Re: automating CF submissions (was xlog location arithmetic)
In-Reply-To: <4F15BBA1.5010701@2ndQuadrant.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox