On Tue, Nov 1, 2011 at 7:21 AM, Fournier, Camille F.
<camille.fourn...@gs.com> wrote:
> Personally, I have no problems with refactorings, but we seem to be spending 
> much of our time dealing with them right now when we really need to get 3.4 
> out the door. It's been months that this release has been going on and 
> splitting the committer attention between 3.4 and refactoring changes feels 
> counterproductive to the needs of the community at this moment.

I second that. There are a few 3.4 blocker issues pending that need
review -- not to mention Mahadev hasn't gotten much feedback on the
release candidate.

There are 35 patch available issues at this moment, most of which are
_not_ refactoring issues. Plenty of patches for committers to review
if they are uninterested in aesthetic issues.

Patrick

>
> -----Original Message-----
> From: Ted Dunning [mailto:ted.dunn...@gmail.com]
> Sent: Monday, October 31, 2011 7:08 PM
> To: dev@zookeeper.apache.org
> Cc: tho...@koch.ro; Benjamin Reed
> Subject: Re: cleanup and subjective patches
>
> How about a middle ground being that aesthetic changes will only be
> considered if they bring significant new testing that helps mitigate the
> stability risk?
>
> On Mon, Oct 31, 2011 at 3:51 PM, Patrick Hunt <ph...@apache.org> wrote:
>
>> EOD this is not a tool problem, it's a resource issue. We have limited
>> committer time available for reviews. IMO Thomas's changes have been
>> pretty mechanical so far, they are easy to review/commit. They do
>> result in some patch churn but I've been surprised it's been so little
>> so far and the code is improving as a result. However we've just
>> scratched the surface (the easy stuff), I do have some concerns as we
>> make more significant structural changes. Personally I think we need
>> to focus on beefing up testing prior to making more significant
>> changes.
>>
>> Ben - We won't get new committers if we limit contributions. We'll
>> just dig a deeper hole for ourselves. Granted if we only review/accept
>> patches of a particular type (or from a particular person) we'll be in
>> a similar situation.
>>
>> Thomas - Ben has highlighted some good points, if we don't discuss
>> these rationally, out in the open, we won't be able to make progress.
>> There are other patches from other contributors that we need to
>> balance the available resources against.
>>
>> http://communityovercode.com/over/
>>
>>
>> Personally: my biggest concern is that we keep releasing solid high
>> quality code that works in production. I'm seeing patches go in that
>> break existing functionality and no one notices. To me the thing we
>> should be focusing on is adding testing. Refactoring and such to
>> improve/increase testing imo is a net positive.
>>
>> Patrick
>>
>> On Mon, Oct 31, 2011 at 2:39 PM, Ted Dunning <ted.dunn...@gmail.com>
>> wrote:
>> > Jumping over to Ben's side of the discussion,
>> >
>> > Git helps with this, but does not eliminate the problem.  At some point
>> the
>> > changes become difficult to understand relative to the new code and may
>> > even collide in ways that are difficult to merge.
>> >
>> > It is true, however, that patches can be kept live using tools like git.
>> >  That is how I kept the multi stuff alive, but there was at least one
>> > tricky merge to be done.
>> >
>> > On Mon, Oct 31, 2011 at 2:35 PM, Thomas Koch <tho...@koch.ro> wrote:
>> >
>> >> Thanks to Ted for replying. I will save the mail I started in the drafts
>> >> folder until I've calmed down again.
>> >>
>> >> Benjamin Reed:
>> >> > deprioritizing them doesn't help because the patches themselves bit
>> >> > rot. shortly they will not apply and then they will be worthless. the
>> >> > poor contributor would then be left with the task of maintaining a
>> >> > patch that would never commit.
>> >> You should really give GIT a try. I've kept a pipeline of half a douzend
>> >> patches filled and current over the last two months while drinking my
>> >> morning
>> >> coffee.
>> >> Likewise have I updated my development branch of over a douzend commits
>> >> every
>> >> morning against the new ZK trunk.
>> >>
>> >> Regards,
>> >>
>> >> Thomas Koch, http://www.koch.ro
>> >>
>> >
>>
>

Reply via email to