(New) reviewer field in Kafka jiras

[Joel Koshy <jj...@gmail.com>]

I had requested infra to add a reviewer field in our jiras -
https://issues.apache.org/jira/browse/INFRA-8189 Hopefully it will
make it easier to formally keep track of a review owner for each jira.

It goes without saying that it should not be interpreted as sole
reviewer - i.e., more than one person can and should review, but I
think this is slightly better than assigning a jira back to a person
to indicate a review is required.

Thanks,

-- 
Joel

Re: (New) reviewer field in Kafka jiras

[Neha Narkhede <ne...@gmail.com>]

Thanks Joel! This is super helpful for distributing the outstanding reviews
amongst all committers and also useful for keeping track of the review todo
list per committer. I'd encourage all active committers to sign up for
patch review as that is one of the most important responsibilities of a
committer and also helps in scaling and building the community.

Currently, we have Apache send out an email for all JIRAs that have patches
outstanding. In addition to that, it will be ideal to send each committer a
list of unresolved JIRAs for which they are listed as a reviewer. I think
this will help provide a better experience to new contributors, who
currently spend significant time pinging committers on individual JIRAs
that can get lost in the list of open source emails.

Thanks,
Neha

On Fri, Aug 29, 2014 at 6:28 AM, Joe Stein <jo...@stealth.ly> wrote:

> Thanks Joel, good stuff
>
> On Fri, Aug 29, 2014 at 2:13 AM, Joel Koshy <jj...@gmail.com> wrote:
>
> > I had requested infra to add a reviewer field in our jiras -
> > https://issues.apache.org/jira/browse/INFRA-8189 Hopefully it will
> > make it easier to formally keep track of a review owner for each jira.
> >
> > It goes without saying that it should not be interpreted as sole
> > reviewer - i.e., more than one person can and should review, but I
> > think this is slightly better than assigning a jira back to a person
> > to indicate a review is required.
> >
> > Thanks,
> >
> > --
> > Joel
> >
>

Re: (New) reviewer field in Kafka jiras

[Joe Stein <jo...@stealth.ly>]

Thanks Joel, good stuff

On Fri, Aug 29, 2014 at 2:13 AM, Joel Koshy <jj...@gmail.com> wrote:

> I had requested infra to add a reviewer field in our jiras -
> https://issues.apache.org/jira/browse/INFRA-8189 Hopefully it will
> make it easier to formally keep track of a review owner for each jira.
>
> It goes without saying that it should not be interpreted as sole
> reviewer - i.e., more than one person can and should review, but I
> think this is slightly better than assigning a jira back to a person
> to indicate a review is required.
>
> Thanks,
>
> --
> Joel
>