> > If you look at the GitHub code reviews > (https://github.com/apache/incubator-spark/pulls), 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: general-unsubscr...@incubator.apache.org For additional commands, e-mail: general-h...@incubator.apache.org
