Received: from localhost (maia-4.hub.org [200.46.204.183]) by postgresql.org (Postfix) with ESMTP id 24D649FB82D for ; Wed, 2 May 2007 09:11:21 -0300 (ADT) Received: from postgresql.org ([200.46.204.71]) by localhost (mx1.hub.org [200.46.204.183]) (amavisd-maia, port 10024) with ESMTP id 18966-04 for ; Wed, 2 May 2007 09:11:17 -0300 (ADT) X-Greylist: from auto-whitelisted by SQLgrey-1.7.4 Received: from momjian.us (momjian.us [70.90.9.53]) by postgresql.org (Postfix) with ESMTP id 7EB1D9FB7B0 for ; Wed, 2 May 2007 09:11:17 -0300 (ADT) Received: (from bruce@localhost) by momjian.us (8.11.6/8.11.6) id l42CBCO23570; Wed, 2 May 2007 08:11:12 -0400 (EDT) From: Bruce Momjian Message-Id: <200705021211.l42CBCO23570@momjian.us> Subject: Re: Feature freeze progress report In-Reply-To: <87k5vrh1wg.fsf@oxford.xeocode.com> To: Gregory Stark Date: Wed, 2 May 2007 08:11:12 -0400 (EDT) CC: Josh Berkus , pgsql-hackers@postgresql.org X-Mailer: ELM [version 2.4ME+ PL123] MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="US-ASCII" X-Virus-Scanned: Maia Mailguard 1.0.1 X-Archive-Number: 200705/110 X-Sequence-Number: 102712 Gregory Stark wrote: > > "Bruce Momjian" writes: > > > We seem to handle trivial patches just fine. > > You keep saying that but I think it's wrong. There are trivial patches that > were submitted last year that are still sitting in the queue. You seem to be looking at something different than me. Which patches? > In fact I claim we handle complex patches better than trivial ones. HOT, LDC, > DSM etc receive tons of feedback and acquire a momentum of their own. > Admittedly GII is a counter-example though. > > Well, I claim it's often the trivial patches that require the domain-specific > knowledge you describe. If they were major patches they would touch more parts > of the system. But that means they should be easy to commit if you could just > fill in the missing knowledge. > > Could you pick a non-committer with the domain-specific knowledge you think a > patch needs and ask for their analysis of the patch then commit it yourself? > You can still review it for general code quality and trust the non-committer's > review of whether the domain-specific change is correct. We are already pushing out patches to people with domain-specific knowledge. Tom posted that summary today. -- Bruce Momjian http://momjian.us EnterpriseDB http://www.enterprisedb.com + If your life is a hard drive, Christ can be your backup. +