Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtp (Exim 4.72) (envelope-from ) id 1U46gX-0003Dz-9X for pgsql-hackers@arkaria.postgresql.org; Sat, 09 Feb 2013 09:25:25 +0000 Received: from localhost ([127.0.0.1] helo=postgresql.org) by malur.postgresql.org with smtp (Exim 4.72) (envelope-from ) id 1U46gW-0006QN-QZ for pgsql-hackers@arkaria.postgresql.org; Sat, 09 Feb 2013 09:25:24 +0000 Received: from makus.postgresql.org ([98.129.198.125]) by malur.postgresql.org with esmtp (Exim 4.72) (envelope-from ) id 1U46gV-0006QI-T3 for pgsql-hackers@postgresql.org; Sat, 09 Feb 2013 09:25:24 +0000 Received: from outmail148104.authsmtp.com ([62.13.148.104]) by makus.postgresql.org with esmtp (Exim 4.72) (envelope-from ) id 1U46gS-00021t-Ih for pgsql-hackers@postgresql.org; Sat, 09 Feb 2013 09:25:23 +0000 Received: from mail-c235.authsmtp.com (mail-c235.authsmtp.com [62.13.128.235]) by punt9.authsmtp.com (8.14.2/8.14.2/Kp) with ESMTP id r199PIAd095440 for ; Sat, 9 Feb 2013 09:25:18 GMT Received: from mail-we0-f175.google.com (mail-we0-f175.google.com [74.125.82.175]) (authenticated bits=0) by mail.authsmtp.com (8.14.2/8.14.2/) with ESMTP id r199PEhJ014266 (version=TLSv1/SSLv3 cipher=RC4-SHA bits=128 verify=FAIL) for ; Sat, 9 Feb 2013 09:25:14 GMT Received: by mail-we0-f175.google.com with SMTP id x8so3672549wey.6 for ; Sat, 09 Feb 2013 01:25:14 -0800 (PST) MIME-Version: 1.0 X-Received: by 10.194.24.42 with SMTP id r10mr14363814wjf.2.1360401914279; Sat, 09 Feb 2013 01:25:14 -0800 (PST) Received: by 10.180.145.195 with HTTP; Sat, 9 Feb 2013 01:25:14 -0800 (PST) In-Reply-To: References: <5112C602.3090109@agliodbs.com> <14838.1360187606@sss.pgh.pa.us> <5112E08C.9040705@commandprompt.com> <511447A4.3070700@agliodbs.com> Date: Sat, 9 Feb 2013 09:25:14 +0000 Message-ID: Subject: Re: Considering Gerrit for CFs From: Simon Riggs To: Magnus Hagander Cc: Josh Berkus , Daniel Farina , "Joshua D. Drake" , Tom Lane , PostgreSQL-development Content-Type: text/plain; charset=ISO-8859-1 X-Server-Quench: 9c26a252-729a-11e2-b5c5-002590a15da7 X-AuthReport-Spam: If SPAM / abuse - report it at: http://www.authsmtp.com/abuse X-AuthRoute: OCdxZQATClZOTQEd DAteCiN5VAwpPBRK HVkIKg5MOFUSTAAU Mk9RLVdJK3UEQkpF VCReGBUITgIzDi11 YhUnOFdCYEpPXBVq WUJLR1hTEEtvBQIA BBsAWhtsdgJDZ3t4 YlljEDJaVU0aAB4H Qk9QF2sAK2NiaWRR WRNacVJWdQFXeR1M O01+VSYPfDcCNX99 R1dpYmFpNGwCJC0K VDsAfxohYW8gPRMG fCVKNjIpBkwJSj8y KRNuBVMGBkcNeks1 KxMKd0lw X-Authentic-SMTP: 61633235383639.1023:706 X-AuthFastPath: 0 (Was 255) X-AuthSMTP-Origin: 74.125.82.175/587 X-AuthVirus-Status: No virus detected - but ensure you scan with your own anti-virus system. X-Pg-Spam-Score: -2.6 (--) List-Archive: List-Help: List-ID: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: X-Mailing-List: pgsql-hackers Precedence: bulk Sender: pgsql-hackers-owner@postgresql.org On 8 February 2013 10:23, Magnus Hagander wrote: >> to solve two problems: >> >> 1. maximize the efficiency of existing reviewer time >> >> 2. make tooling not be an obstacle to getting new reviewers > > I think you are missing a fundamental part in this - which is "0. > don't negatively affect the efficiency of existing committer time". Agreed. Process improvement is very important, but not more important than the process itself. What is the developer meeting for if not this discussion? Let's do some patch reviews and commits now and save the lets-make-it-better stufff for later. -- Simon Riggs http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers