[DISCUSS] Patch available backlog

[Flavio Junqueira <fp...@yahoo.com>]

I was checking the issues on jira and I realized that we have currently 76 patch available issues. This is a fairly large number and I'd like some help to get that number down. One way is to split issues among the folks willing to review patches. It would be great to have committers signing up to review patches, but we can definitely get contributors helping with the reviews and I actually encourage reviews from contributors as well. 

To split the load, one idea is to set up an hour or two for us to be simultaneously on irc to discuss the split and any contention on issues. But, if this idea doesn't work, I would be very happy to hear alternatives. Ultimately I'm interested in making progress with the patches submitted.

If you agree with the idea of being on irc, please share some slots that suit you.

Thanks,
-Flavio

Re: [DISCUSS] Patch available backlog

[German Blanco <ge...@gmail.com>]

Hello,
although I am not sure whether I can be considered a "contributor", since I
haven't added anything yet, I am willing to review patches if it is of any
help.
I have been looking at the code for some months now, specially the server
part.
Irc is ok with me.
Monday and Tuesday next week I will be busy during CET 8 to 18. Otherwise
any time (although I understand that my attendance may not be that relevant
:-)
Regards,
Germán.


On Thu, May 9, 2013 at 11:41 PM, Flavio Junqueira <fp...@yahoo.com>wrote:

> I was checking the issues on jira and I realized that we have currently 76
> patch available issues. This is a fairly large number and I'd like some
> help to get that number down. One way is to split issues among the folks
> willing to review patches. It would be great to have committers signing up
> to review patches, but we can definitely get contributors helping with the
> reviews and I actually encourage reviews from contributors as well.
>
> To split the load, one idea is to set up an hour or two for us to be
> simultaneously on irc to discuss the split and any contention on issues.
> But, if this idea doesn't work, I would be very happy to hear alternatives.
> Ultimately I'm interested in making progress with the patches submitted.
>
> If you agree with the idea of being on irc, please share some slots that
> suit you.
>
> Thanks,
> -Flavio

RE: [DISCUSS] Patch available backlog

[FPJ <fp...@yahoo.com>]

Thanks for the feedback so far. The specific patches you people brought up
are fine with me, they look like important new features and I'd be happy to
have them in. I'm more concerned about the backlog problem in general,
though. There are patches that have been sitting there for a long time and
it would be nice to get to a resolution of what to do with them, not
necessarily committing.

The irc idea was simply to make it easier for people to coordinate on what
to review and such, but it sounds like it didn't get a lot of traction, so I
suggest we don't do that for now. I'll try to use irc more, though, in the
case anyone is interested in a discussion. For the ones willing to do
reviews, I'd suggest  to go over the list of patch availables on jira and
pick the ones you'd feel comfortable reviewing. This is a hint more for the
people new to the community like German, since many people here already know
the procedure.

I'll also go over the list and poke people explicitly, I hope you don't
mind. 

-Flavio

-----Original Message-----
From: Edward Ribeiro [mailto:edward.ribeiro@gmail.com] 
Sent: 13 May 2013 18:02
To: dev@zookeeper.apache.org
Subject: Re: [DISCUSS] Patch available backlog

Agree with Patrick and, in my opinion, Thawan's "local session" patch and
Skye's jetty support should be high priority on list. After Alex's dynamic
config, those two patches should be evaluated asap.

I have been reviewing both patches at Review Board, as far as my n00b
knowledge of ZK inner works permits, but I'd love to see more patches at
review board! I am glad to help so let's get it started. Count me in!

Edward Ribeiro


On Fri, May 10, 2013 at 12:57 PM, Benjamin Reed <br...@apache.org> wrote:

> tues-thurs evenings next week would work for me.
>
> ben
>
>
> On Fri, May 10, 2013 at 8:53 AM, Patrick Hunt <ph...@apache.org> wrote:
>
> > On Thu, May 9, 2013 at 2:41 PM, Flavio Junqueira 
> > <fp...@yahoo.com>
> > wrote:
> > > I was checking the issues on jira and I realized that we have 
> > > currently
> > 76 patch available issues. This is a fairly large number and I'd 
> > like
> some
> > help to get that number down. One way is to split issues among the 
> > folks willing to review patches. It would be great to have 
> > committers signing
> up
> > to review patches, but we can definitely get contributors helping 
> > with
> the
> > reviews and I actually encourage reviews from contributors as well.
> > >
> >
> > I had been reviewing them at least every month. However with the 
> > recent big patches going in (scalability changes, dynamic
> > reconfig,...) I held off to not impact/delay those patches. I've 
> > been super busy at home/work but I'll try to find some time, It 
> > would be great if other folks jumped in. They key I've found is to 
> > set aside some time and just focus on crunching through. I'd love to 
> > see 3.4.6 and 3.5.0 releases soon. We have some good fixes for 
> > 3.5.6, and there are only a couple of major 3.5.0 issues to go, if 
> > we could get Thawan's "local session" patch and Skye's jetty support 
> > in, as well as a few high priority 3.5 fixes (1324 in particular) 
> > I'd love to cut a beta release candidate for 3.5.
> >
> > > To split the load, one idea is to set up an hour or two for us to 
> > > be
> > simultaneously on irc to discuss the split and any contention on issues.
> > But, if this idea doesn't work, I would be very happy to hear
> alternatives.
> > Ultimately I'm interested in making progress with the patches submitted.
> > >
> >
> > This is a good idea, but I find that just a single person reviewing 
> > the patch is often sufficient. Once you boil things down a bit (I typ.
> > find that ~30% of the patches are pretty simple and easy to commit) 
> > it's good to have a couple folks focus on the tougher/remaining 
> > issues. Esp if it involves a "feature" level change.
> >
> > > If you agree with the idea of being on irc, please share some 
> > > slots
> that
> > suit you.
> > >
> >
> > I'm almost always on email or im. FF to ping me if you need to 
> > discuss something.
> >
> > Patrick
> >
>

Re: [DISCUSS] Patch available backlog

[Edward Ribeiro <ed...@gmail.com>]

Agree with Patrick and, in my opinion, Thawan's "local session" patch and
Skye's jetty support should be high priority on list. After Alex's dynamic
config, those two patches should be evaluated asap.

I have been reviewing both patches at Review Board, as far as my n00b
knowledge of ZK inner works permits, but I'd love to see more patches at
review board! I am glad to help so let's get it started. Count me in!

Edward Ribeiro


On Fri, May 10, 2013 at 12:57 PM, Benjamin Reed <br...@apache.org> wrote:

> tues-thurs evenings next week would work for me.
>
> ben
>
>
> On Fri, May 10, 2013 at 8:53 AM, Patrick Hunt <ph...@apache.org> wrote:
>
> > On Thu, May 9, 2013 at 2:41 PM, Flavio Junqueira <fp...@yahoo.com>
> > wrote:
> > > I was checking the issues on jira and I realized that we have currently
> > 76 patch available issues. This is a fairly large number and I'd like
> some
> > help to get that number down. One way is to split issues among the folks
> > willing to review patches. It would be great to have committers signing
> up
> > to review patches, but we can definitely get contributors helping with
> the
> > reviews and I actually encourage reviews from contributors as well.
> > >
> >
> > I had been reviewing them at least every month. However with the
> > recent big patches going in (scalability changes, dynamic
> > reconfig,...) I held off to not impact/delay those patches. I've been
> > super busy at home/work but I'll try to find some time, It would be
> > great if other folks jumped in. They key I've found is to set aside
> > some time and just focus on crunching through. I'd love to see 3.4.6
> > and 3.5.0 releases soon. We have some good fixes for 3.5.6, and there
> > are only a couple of major 3.5.0 issues to go, if we could get
> > Thawan's "local session" patch and Skye's jetty support in, as well as
> > a few high priority 3.5 fixes (1324 in particular) I'd love to cut a
> > beta release candidate for 3.5.
> >
> > > To split the load, one idea is to set up an hour or two for us to be
> > simultaneously on irc to discuss the split and any contention on issues.
> > But, if this idea doesn't work, I would be very happy to hear
> alternatives.
> > Ultimately I'm interested in making progress with the patches submitted.
> > >
> >
> > This is a good idea, but I find that just a single person reviewing
> > the patch is often sufficient. Once you boil things down a bit (I typ.
> > find that ~30% of the patches are pretty simple and easy to commit)
> > it's good to have a couple folks focus on the tougher/remaining
> > issues. Esp if it involves a "feature" level change.
> >
> > > If you agree with the idea of being on irc, please share some slots
> that
> > suit you.
> > >
> >
> > I'm almost always on email or im. FF to ping me if you need to discuss
> > something.
> >
> > Patrick
> >
>

Re: [DISCUSS] Patch available backlog

[Benjamin Reed <br...@apache.org>]

tues-thurs evenings next week would work for me.

ben


On Fri, May 10, 2013 at 8:53 AM, Patrick Hunt <ph...@apache.org> wrote:

> On Thu, May 9, 2013 at 2:41 PM, Flavio Junqueira <fp...@yahoo.com>
> wrote:
> > I was checking the issues on jira and I realized that we have currently
> 76 patch available issues. This is a fairly large number and I'd like some
> help to get that number down. One way is to split issues among the folks
> willing to review patches. It would be great to have committers signing up
> to review patches, but we can definitely get contributors helping with the
> reviews and I actually encourage reviews from contributors as well.
> >
>
> I had been reviewing them at least every month. However with the
> recent big patches going in (scalability changes, dynamic
> reconfig,...) I held off to not impact/delay those patches. I've been
> super busy at home/work but I'll try to find some time, It would be
> great if other folks jumped in. They key I've found is to set aside
> some time and just focus on crunching through. I'd love to see 3.4.6
> and 3.5.0 releases soon. We have some good fixes for 3.5.6, and there
> are only a couple of major 3.5.0 issues to go, if we could get
> Thawan's "local session" patch and Skye's jetty support in, as well as
> a few high priority 3.5 fixes (1324 in particular) I'd love to cut a
> beta release candidate for 3.5.
>
> > To split the load, one idea is to set up an hour or two for us to be
> simultaneously on irc to discuss the split and any contention on issues.
> But, if this idea doesn't work, I would be very happy to hear alternatives.
> Ultimately I'm interested in making progress with the patches submitted.
> >
>
> This is a good idea, but I find that just a single person reviewing
> the patch is often sufficient. Once you boil things down a bit (I typ.
> find that ~30% of the patches are pretty simple and easy to commit)
> it's good to have a couple folks focus on the tougher/remaining
> issues. Esp if it involves a "feature" level change.
>
> > If you agree with the idea of being on irc, please share some slots that
> suit you.
> >
>
> I'm almost always on email or im. FF to ping me if you need to discuss
> something.
>
> Patrick
>

Re: [DISCUSS] Patch available backlog

[Patrick Hunt <ph...@apache.org>]

On Thu, May 9, 2013 at 2:41 PM, Flavio Junqueira <fp...@yahoo.com> wrote:
> I was checking the issues on jira and I realized that we have currently 76 patch available issues. This is a fairly large number and I'd like some help to get that number down. One way is to split issues among the folks willing to review patches. It would be great to have committers signing up to review patches, but we can definitely get contributors helping with the reviews and I actually encourage reviews from contributors as well.
>

I had been reviewing them at least every month. However with the
recent big patches going in (scalability changes, dynamic
reconfig,...) I held off to not impact/delay those patches. I've been
super busy at home/work but I'll try to find some time, It would be
great if other folks jumped in. They key I've found is to set aside
some time and just focus on crunching through. I'd love to see 3.4.6
and 3.5.0 releases soon. We have some good fixes for 3.5.6, and there
are only a couple of major 3.5.0 issues to go, if we could get
Thawan's "local session" patch and Skye's jetty support in, as well as
a few high priority 3.5 fixes (1324 in particular) I'd love to cut a
beta release candidate for 3.5.

> To split the load, one idea is to set up an hour or two for us to be simultaneously on irc to discuss the split and any contention on issues. But, if this idea doesn't work, I would be very happy to hear alternatives. Ultimately I'm interested in making progress with the patches submitted.
>

This is a good idea, but I find that just a single person reviewing
the patch is often sufficient. Once you boil things down a bit (I typ.
find that ~30% of the patches are pretty simple and easy to commit)
it's good to have a couple folks focus on the tougher/remaining
issues. Esp if it involves a "feature" level change.

> If you agree with the idea of being on irc, please share some slots that suit you.
>

I'm almost always on email or im. FF to ping me if you need to discuss
something.

Patrick