Received: from localhost (unknown [200.46.204.183]) by postgresql.org (Postfix) with ESMTP id CC6F22E2EF9 for ; Fri, 7 Mar 2008 15:58:24 -0400 (AST) Received: from postgresql.org ([200.46.204.71]) by localhost (mx1.hub.org [200.46.204.183]) (amavisd-maia, port 10024) with ESMTP id 82975-04 for ; Fri, 7 Mar 2008 15:58:22 -0400 (AST) X-Greylist: from auto-whitelisted by SQLgrey-1.7.5 Received: from mail01.enterprisedb.com (mail01.enterprisedb.com [63.246.7.168]) by postgresql.org (Postfix) with ESMTP id F02ED2E2EAE for ; Fri, 7 Mar 2008 15:58:21 -0400 (AST) thread-index: AciAja0jOQTbGVNDRaWfIbZZLIQGEA== Received: from [192.168.1.34] ([86.0.99.82]) by mail01.enterprisedb.com over TLS secured channel with Microsoft SMTPSVC(6.0.3790.3959); Fri, 7 Mar 2008 14:58:56 -0500 Content-Class: urn:content-classes:message Importance: normal X-MimeOLE: Produced By Microsoft MimeOLE V6.00.3790.4133 Message-ID: <47D19E18.4060303@enterprisedb.com> Date: Fri, 07 Mar 2008 19:57:12 +0000 From: "Heikki Linnakangas" Organization: EnterpriseDB User-Agent: Mozilla-Thunderbird 2.0.0.9 (X11/20080110) MIME-Version: 1.0 To: "Alvaro Herrera" Cc: "Joshua D. Drake" , "PostgreSQL-development" Subject: Re: Commitfest process References: <47D1521E.8090606@enterprisedb.com> <20080307101413.7d191e52@commandprompt.com> <47D18D80.2050107@enterprisedb.com> <20080307105047.2e34a156@commandprompt.com> <20080307194612.GB16862@alvh.no-ip.org> In-Reply-To: <20080307194612.GB16862@alvh.no-ip.org> Content-Type: text/plain; format=flowed; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit X-OriginalArrivalTime: 07 Mar 2008 19:58:56.0296 (UTC) FILETIME=[AD19EA80:01C8808D] X-Virus-Scanned: Maia Mailguard 1.0.1 X-Archive-Number: 200803/199 X-Sequence-Number: 115000 Alvaro Herrera wrote: > Joshua D. Drake wrote: > >> On Fri, 07 Mar 2008 18:46:24 +0000 >> "Heikki Linnakangas" wrote: >> >>> I think we'll have more success convincing patch authors to update a >>> wiki page, than we'll have to convince reviewers to do so. I know >>> that's true at least for me. If I want people to review my patch, I'm >>> ready to sing and dance if that's what it takes. But if there's extra >>> steps in reviewing a patch, I might just not bother. >> Well that is what my email is about, dropping extra steps :). > > I agree that that's a good objective, but I think a Wiki makes for a > crappy patch tracker. Sure, but let's not turn this into a bug/patch tracker discussion, please :-/. A wiki is not ideal, but it's there. The main point of my proposal is: let's make the *authors* who want their stuff to be reviewed as part of a commitfest do the extra work. There would be no extra work required for patch reviewers. Sure, we can refine that later. making it easier for patch authors as well, but I don't think it's an unreasonable amount of work to keep one line per patch up-to-date in a wiki. The line doesn't need to contain anything else than title of patch, name of the author, and links to latest patch and relevant discussion threads, if applicable. And commitfests are short, you only need to update the wiki a couple of times during the commitfest. -- Heikki Linnakangas EnterpriseDB http://www.enterprisedb.com