incubator-general mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Craig L Russell <>
Subject Re: [VOTE] Graduation of Apache Spark from the Incubator
Date Thu, 06 Feb 2014 17:28:28 GMT
Having developer/committer comments from a github review automatically show up on dev@ lists
will ease my concern that the git activities somehow avoid the "if it isn't on the mail lists,
it didn't happen" requirement.


On Feb 6, 2014, at 9:08 AM, Michael Joyce wrote:

> How is doing a review on Github any different than doing a review on Review
> Board? If there's concern that work on Github isn't being adequately
> mirrored on the mailing lists then that sounds like an Infra problem to me.
> There are plenty of hooks [1] that makes this easy to do and, if I'm not
> mistaken, pull requests through Github are already supposed to mail dev@. I
> would assume that any comment on them should as well.
> [1]
> -- Joyce
> On Thu, Feb 6, 2014 at 1:22 AM, David Nalley <> wrote:
>>> If you look at the GitHub code reviews (
>>, you'll see that lots of
>> people are contributing to reviewing. But I agree that the new committer
>> onboarding process should include having them do a test commit.
>> Matei:
>> Reviews being done on github causes me a bit of concern. Generally
>> dev@ should be the nexus for a project, and most of the projects that
>> have github in the workflow ensure that pull requests get sent to dev@
>> so the discussions, reviews, and patches themselves are seen and
>> archived there.
>> I see that one of your mentors brought up similar concerns about this
>> (as well as where discussions on strategy or roadmap happen) yesterday
>> on dev@ - that doesn't leave me with warm fuzzy assurance that the
>> project groks the Apache way.
>> --David
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail:
>> For additional commands, e-mail:

Craig L Russell
Architect, Oracle
408 276-5638
P.S. A good JDO? O, Gasp!

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message