> > We can do both of course, but I think apache infra already has > what is needed for this.
Yes, if Apache INFRA has this, great. I would pick something that is a *push* mechanism and leave the *pull* as an option for proactive folks to refer to. Honestly, today you can go and save the JIRA query that lists you as a reviewer today, but doesn't look like that is something people are doing. I'll be honest, I don't remember to do that myself. I would really encourage people to make this an email that gets sent out to the entire dev@ mailing list so everyone is on the loop about the patches that need reviews, who is on-point and which ones have no reviewers. On Wed, Apr 8, 2015 at 10:22 AM, Joel Koshy <jjkosh...@gmail.com> wrote: > The weekly email sounds good, but I would much rather if that were > integrated into the wiki/jira system than run a custom script on > non-apache infra. We did something similar during 0.8 replication > development where we embedded JQL links in the wiki and had a nice > dashboard view of jiras on a wiki page. > > So I was thinking of something similar - i.e., a wiki page of > committers and contributors that have signed up to review patches and > have a jira table for each. We could allocate no more than five > minutes at the hangout we have been doing to go over the top > patches per reviewer and decide whether patch reviews need to move > around. I think having this view helps in the interest of > visibility. > > We can do both of course, but I think apache infra already has > what is needed for this. > > Thanks, > > Joel > > On Wed, Apr 08, 2015 at 09:07:00AM -0700, Neha Narkhede wrote: > > The other suggestion I'd like to make involves writing a simple JIRA > script > > (whaat?). Currently the weekly email we get from JIRA just tells you what > > JIRAs have patches. But it doesn't tell you which committers/reviewers > have > > signed up to review those patches. The 2nd piece of information can be > > really helpful. Here's why- > > > > I'm sure everyone wants to know which patches are open and since our > > community is so active, that is often a long list even if each JIRA is > > actively being reviewed. As a busy committer, I think if I knew the list > of > > JIRAs I've signed up to review and commit, I would really appreciate the > > weekly reminder. It allows me to either get to it or give it up to > someone > > else. > > > > If someone had some time, they can look at my patch-review script that > uses > > some of JIRA's python APIs and write a little script that does this. At > > Confluent, we are happy to host that script on some EC2 machine. > > > > Thanks, > > Neha > > > > On Wed, Apr 8, 2015 at 8:41 AM, Gwen Shapira <gshap...@cloudera.com> > wrote: > > > > > Hi Jun, > > > > > > Do reviewers have to be committers? > > > I'll be happy to review patches in areas that I'm familiar with, if > this > > > makes life easier for committers and helps us get improvements in > faster. > > > > > > Gwen > > > > > > On Tue, Apr 7, 2015 at 11:10 PM, Jun Rao <j...@confluent.io> wrote: > > > > > > > Hi, everyone, > > > > > > > > Currently, we have accumulated quite a few jira backlogs. We can > probably > > > > brainstorm a bit what we can do to improve this. A few thoughts. > > > > > > > > 1. There are quite a few jiras marked with "patch available", but > with > > > > unaddressed review comments. I made a pass and moved them back to "in > > > > progress" (just click the "Resume Progress") button. Now, there are > about > > > > 100 jiras with valid patches. As people review the patches, it may be > > > > useful to keep the status updated as well (thanks Neha for the > > > suggestion). > > > > > > > > 2. Some of the jiras have trivial patches. Perhaps we can just assign > > > them > > > > with "trivial" priority. This way, a committer with only limited > amount > > > of > > > > time may be able to help out. > > > > > > > > 3. Some of the jiras have been tagged with reviewers. It would be > good if > > > > those reviewers can spend time finishing the reviews. If they can't > do > > > the > > > > review soon, they can unset the reviewer field so that someone else > can > > > > pick up the review. > > > > > > > > 4. Perhaps we can also spend a bit of time to triage at least the > active > > > > jiras during our weekly hangout. > > > > > > > > Anything else that we can do? > > > > > > > > Thanks, > > > > > > > > Jun > > > > > > > > > > > > > > > -- > > Thanks, > > Neha > > -- Thanks, Neha
