Spark also has over 400 PRs open :) Taking a quick look, we have ones like this: https://github.com/apache/spark/pull/20161
Which have been open for 2 weeks with no reviews (even with an @mention) in the comments. I think it's a shame that the GitHub PR model doesn't allow picking reviewers, especially since it negates their very own filters. I suppose we'll just have to use the @mention for now. On Jan 17, 2018, at 3:08 PM, Vivek Ratnavel <[email protected]<mailto:[email protected]>> wrote: I have seen some other very active Apache projects including Apache Spark using this model successfully for a long time now. They have a portal to keep track of open PRs and stale PRs categorized by components - https://spark-prs.appspot.com/ The contributors open PRs and wait for the community to review it. For instance, take a look at this - https://github.com/apache/spark/pull/20297 On Wed, Jan 17, 2018 at 12:00 PM, Jonathan Hurley <[email protected]> wrote: We have a very active project with potentially dozens of new PRs coming in every day - I don't think relying on the community to look is a very sustainable model. Is there no way to add reviewers without tagging them? There seems to be a way to filter by reviews needing attention - seems silly that non-committers can't add committers ... On Jan 17, 2018, at 2:51 PM, Vivek Ratnavel <[email protected]> wrote: Hi Jason, Thanks for bringing this up. For contributors, the best way to notify people would be to tag them in comments. This is another good reason to add [component] tags to the PR title, so that committers who are working on those components can pro-actively take a look at the newly opened pull requests. The committers of the community should actively check newly created PRs and resolve them. If the PR is not reviewed by any committer for a long time, then the contributor can try pinging the committers by tagging them in comments as I mentioned before. Relevant committers can be found by doing a git blame on the files contributed in the patch. Thanks, Vivek Ratnavel On Wed, Jan 17, 2018 at 11:33 AM, Jason Golieb <[email protected]> wrote: As a non-committer, is it possible to add reviewers to a pull request? It doesn’t look like it to me. What’s the suggested procedure for getting eyes on my PRs? -- -Vivek Ratnavel S
