Thanks for all the +1s! I have created https://issues.apache.org/jira/browse/SAMZA-880 to track it.
Thanks! -Yi On Wed, Feb 24, 2016 at 5:31 PM, Boris Shkolnik <bor...@gmail.com> wrote: > +1 for pull requests. > > On Thu, Feb 18, 2016 at 3:53 PM, Yi Pan <nickpa...@gmail.com> wrote: > > > Hi, all, > > > > I want to start the discussion on our code review/commit process. > > > > I felt that our code review and check-in process is a little bit > > cumbersome: > > - developers need to create RBs and attach diff to JIRA > > - committers need to review RBs, dowload diff and apply, then push. > > > > It would be much lighter if we take the pull request only approach, as > > Kafka already converted to: > > - for the developers, the only thing needed is to open a pull request. > > - for committers, review and apply patch is from the same PR and merge > can > > be done directly on remote git repo. > > > > Of course, there might be some hookup scripts that we will need to link > > JIRA w/ pull request in github, which Kafka already does. Any comments > and > > feedbacks are welcome! > > > > Thanks! > > > > -Yi > > >
