public inbox for [email protected]
help / color / mirror / Atom feedFrom: Josh Berkus <[email protected]>
To: Magnus Hagander <[email protected]>
Cc: PostgreSQL-development <[email protected]>
Subject: Re: Considering Gerrit for CFs
Date: Wed, 06 Feb 2013 13:25:31 -0800
Message-ID: <[email protected]> (raw)
In-Reply-To: <CABUevEyEyYZ58717QeVutJ0bgiN1fKDb0EEObVNax0+cZQRdJA@mail.gmail.com>
References: <[email protected]>
<CABUevEyEyYZ58717QeVutJ0bgiN1fKDb0EEObVNax0+cZQRdJA@mail.gmail.com>
List-Unsubscribe: <mailto:[email protected]?body=unsub%20pgsql-hackers>
> This is probably not something we should discuss right now - it's
> better discussed when we're not right inthe middle of a commitfest,
> no?
Well, *if* we were to change tooling, the time to do it would be during
beta. Hence, bringing it up now.
> We have no ad-hoc PHP, but I'm assume you're referring to the cf
> management app that's in perl?
Sorry, "ad-hoc perl".
>> rather than email-centric, so it would modify our current email-centric
>> workflow (e.g. reviews are posted via a special git commit). Unlike
>
> Previously, we've said we do not want to do this. And I think in
> general, it's a realliy bad idea to have a tool dictate the workflow.
> It should be the other way around.
Yeah. It would be theoretically possible to change things so that
Gerrit would accept email reviews, but anyone who's worked with RT can
tell you that automated processing of email is error-prone. That's why
nobody does it. Note that Gerrit is perfectly capable of *sending*
email to the list, it's just *receiving* it which is an issue.
Mind you, when I explained our current CF review workflow for the SF
ReviewFest last year, the attendees thought I was insane. It's kept me
from doing more reviewfests. Our current workflow and tooling is
definitely a serious obstacle to gettng more reviewers. Seems like a
good topic for the developer meeting.
>> The advantages in features would be substantial: a better interface,
>> ways to perform automated tasks (like remind submitters that a patch is
>> waiting on author), online diffs, automated testing integration, and a
>> configurable review workflow process.
>
> Could you point to an example somewhere that we could check such features out?
This is a good place to start: https://review.openstack.org
>> The existing Gerrit community would be keen to have the PostgreSQL
>> project as a major user, though, and would theoretically help with
>> modification needs. Current major users are OpenStack, Mediawiki,
>> LibreOffice and QT.
>
> "theoretically"?
Well, I think we learned from Gforge that folks are more willing to
promise help than to deliver it.
> I just took a quick look at their system, and when they start talking
> about requirements in the 100's of Gb of RAM, 24 core machines and
> SSD, I get scared :) But that's to "scale" it - doesn't mention when
> you need to do anything like that. I'm assuming we'd be tiny.
Yeah, I have no idea. Given that it's Java, I'd assume that
requirements would go up.
--
Josh Berkus
PostgreSQL Experts Inc.
http://pgexperts.com
--
Sent via pgsql-hackers mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
view thread (35+ messages) latest in thread
reply
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: [email protected]
Cc: [email protected], [email protected], [email protected]
Subject: Re: Considering Gerrit for CFs
In-Reply-To: <[email protected]>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox