Re: [DISCUSS] Removing or changing maintainer process

2016-06-01 Thread Nicholas Chammas
I just heard about mention-bot at PyCon 2016 : https://github.com/facebook/mention-bot Do you have a GitHub project that is too big for people to subscribe to all the notifications? The mention bot will automatically mention potential reviewers on

Re: [DISCUSS] Removing or changing maintainer process

2016-05-21 Thread Steve Loughran
> On 19 May 2016, at 16:34, Matei Zaharia wrote: > > Hi folks, > > Around 1.5 years ago, Spark added a maintainer process for reviewing API and > architectural changes >

Re: [DISCUSS] Removing or changing maintainer process

2016-05-19 Thread Andrew Or
+1, some maintainers are hard to find 2016-05-19 9:03 GMT-07:00 Imran Rashid : > +1 (binding) on removal of maintainers > > I dont' have a strong opinion yet on how to have a system for finding the > right reviewers. I agree it would be nice to have something to help you >

Re: [DISCUSS] Removing or changing maintainer process

2016-05-19 Thread Imran Rashid
+1 (binding) on removal of maintainers I dont' have a strong opinion yet on how to have a system for finding the right reviewers. I agree it would be nice to have something to help you find reviewers, though I'm a little skeptical of anything automatic. On Thu, May 19, 2016 at 10:34 AM, Matei

Re: [DISCUSS] Removing or changing maintainer process

2016-05-19 Thread Nicholas Chammas
I’ve also heard that we should try to keep some other instructions for contributors to find the “right” reviewers, so it would be great to see suggestions on that. For my part, I’d personally prefer something “automatic”, such as easily tracking who reviewed each patch and having people look at

Re: [DISCUSS] Removing or changing maintainer process

2016-05-19 Thread Tom Graves
+1 (binding) Tom On Thursday, May 19, 2016 10:35 AM, Matei Zaharia wrote: Hi folks, Around 1.5 years ago, Spark added a maintainer process for reviewing API and architectural changes

Re: [DISCUSS] Removing or changing maintainer process

2016-05-19 Thread Mridul Muralidharan
+1 (binding) on removing maintainer process. I agree with your opinion of "automatic " instead of a manual list. Regards Mridul On Thursday, May 19, 2016, Matei Zaharia wrote: > Hi folks, > > Around 1.5 years ago, Spark added a maintainer process for reviewing API >

[DISCUSS] Removing or changing maintainer process

2016-05-19 Thread Matei Zaharia
Hi folks, Around 1.5 years ago, Spark added a maintainer process for reviewing API and architectural changes (https://cwiki.apache.org/confluence/display/SPARK/Committers#Committers-ReviewProcessandMaintainers) to make sure these are seen by people who spent a lot of time on that component.