X-Original-To: pgsql-hackers-postgresql.org@postgresql.org Received: from localhost (mx1.hub.org [200.46.208.251]) by postgresql.org (Postfix) with ESMTP id 126D19FB52C for ; Sat, 2 Sep 2006 13:18:47 -0300 (ADT) Received: from postgresql.org ([200.46.204.71]) by localhost (mx1.hub.org [200.46.208.251]) (amavisd-new, port 10024) with ESMTP id 26360-02-8 for ; Sat, 2 Sep 2006 13:18:36 -0300 (ADT) X-Greylist: from auto-whitelisted by SQLgrey- Received: from lists.commandprompt.com (host-130.commandprompt.net [207.173.203.130]) by postgresql.org (Postfix) with ESMTP id E34199FB50F for ; Sat, 2 Sep 2006 13:11:09 -0300 (ADT) Received: from [192.168.1.50] (or-67-76-146-141.sta.embarqhsd.net [67.76.146.141]) (authenticated bits=0) by lists.commandprompt.com (8.13.7/8.13.6) with ESMTP id k82GAr9g029734 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Sat, 2 Sep 2006 09:10:54 -0700 Message-ID: <44F9AD1A.5070802@commandprompt.com> Date: Sat, 02 Sep 2006 09:11:06 -0700 From: "Joshua D. Drake" Organization: Command Prompt, Inc. User-Agent: Thunderbird 1.5.0.5 (X11/20060728) MIME-Version: 1.0 To: Tom Lane CC: Theo Schlossnagle , Alvaro Herrera , Peter Eisentraut , pgsql-hackers@postgresql.org, Josh Berkus , Bruce Momjian Subject: Re: Getting a move on for 8.2 beta References: <200609012225.k81MPwJ02170@momjian.us> <200609011639.42280.josh@agliodbs.com> <26043.1157154656@sss.pgh.pa.us> <200609020209.36483.peter_e@gmx.net> <20060902014835.GB14288@alvh.no-ip.org> <3246.1157210938@sss.pgh.pa.us> <3573.1157212990@sss.pgh.pa.us> In-Reply-To: <3573.1157212990@sss.pgh.pa.us> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Virus-Scanned: ClamAV version 0.88.3, clamav-milter version 0.88.3 on projects.commandprompt.com X-Virus-Status: Clean X-Greylist: Sender succeded SMTP AUTH authentication, not delayed by milter-greylist-1.6 (lists.commandprompt.com [192.168.2.159]); Sat, 02 Sep 2006 09:10:55 -0700 (PDT) X-Virus-Scanned: Maia Mailguard 1.0.1 X-Spam-Status: No, hits=0.201 tagged_above=0 required=5 tests=AWL, FORGED_RCVD_HELO X-Spam-Level: X-Archive-Number: 200609/145 X-Sequence-Number: 89674 Tom Lane wrote: > Theo Schlossnagle writes: >> Additionally, what problem is accepting incremental patches supposed >> to solve? > > Keeping the individual patches reviewable is one useful goal. > > We may be talking at cross-purposes here. The sort of thing I think > Alvaro is imagining is something like what I did a year or two back when > I wanted to make the executor treat plan trees as read-only --- if > memory serves, I did that in three or four commits spread over a week or > two. To second this, Alvaro is constantly beating our (cmd) other developers to do this, so I would guess that you are correct :). I find also that this method allows someone like me, who can read C and understand good parts of it to get the gist of what is going on without trying to grok the whole thing. Large patches make it very difficult. Sincerely, Joshua D. Drake -- === The PostgreSQL Company: Command Prompt, Inc. === Sales/Support: +1.503.667.4564 || 24x7/Emergency: +1.800.492.2240 Providing the most comprehensive PostgreSQL solutions since 1997 http://www.commandprompt.com/