From other communities, there isn't really ownership over any part of the project, so anyone is free to review any patch. If you decide to review a patch and believe that a specific person should have a look to support your recommendation because that person has more experience, just ping the person in the jira. It is also not uncommon that committers ping each other for a second opinion, but again anyone should be able to review any patch as I see it.
-Flavio > On 11 Aug 2015, at 21:50, Grant Henke <ghe...@cloudera.com> wrote: > >> >> 2. Encourage contributors to set the "reviewer" field when change JIRA >> status to "patch available", and encourage volunteers assigning themselves >> to "reviewers" for pending tickets. > > > Is there somewhere that describes who to pick as a reviewer based on the > patch? Would it be worth listing volunteer reviews in a similar location? > > On Tue, Aug 11, 2015 at 2:14 PM, Guozhang Wang <wangg...@gmail.com> wrote: > >> First of all, WebEx seems working! And we will upload the recorded video >> later. >> >> Quick summary: >> >> KIP-26: RP-99 (https://github.com/apache/kafka/pull/99) pending for >> reviews. >> >> KIP-28: RP-130 (https://github.com/apache/kafka/pull/130) looking for >> feedbacks on: >> >> 1. API design (see o.k.a.stream.examples). >> 2. Architecture design (see KIP wiki page) >> 3. Packaging options. >> >> KIP-29: we will do a quick fix for unblocking production issues with >> hard-coded interval values, while at the same time keep the KIP open for >> further discussions about end state configurations. >> >> KIP-4: KAFKA-1695 / 2210 pending for reviews. >> >> Review Backlog Management: >> >> 1. Remind people to change JIRA status as "patch available" when they >> contribute the patch, and change the status back to "in progress" after it >> is reviewed, as indicated in: >> >> https://cwiki.apache.org/confluence/display/KAFKA/Contributing+Code+Changes >> >> 2. Encourage contributors to set the "reviewer" field when change JIRA >> status to "patch available", and encourage volunteers assigning themselves >> to "reviewers" for pending tickets. >> >> -- Guozhang >> > > > > -- > Grant Henke > Software Engineer | Cloudera > gr...@cloudera.com | twitter.com/gchenke | linkedin.com/in/granthenke
