Mailing-List: contact dev-help@directory.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Apache Directory Developers List" <dev@directory.apache.org>
Received-SPF: pass (athena.apache.org: domain of ayaselcuk@gmail.com
 designates 74.125.82.178 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CADwPi+GhOYNucMsTZETuHtWne7sDB0Zei00xyggqxvPfSbtn=Q@mail.gmail.com>
References: 
 <CALhXAG1RjUzwxufMG5RyjEQ9eH1y1Jkom_EC3n=NWLRp+_228w@mail.gmail.com>
	<CABzFU-dAO2Rg0fXSVn1bQy2PhySM0qje4S_cO141bxp0+5arEA@mail.gmail.com>
	<CADwPi+GhOYNucMsTZETuHtWne7sDB0Zei00xyggqxvPfSbtn=Q@mail.gmail.com>
Date: Wed, 14 Mar 2012 14:52:01 -0700
Message-ID: 
 <CALhXAG0zYL_6DzpjwxEzxDqEEt+v+PJzqb8LZrzVYcuFyV6Rfg@mail.gmail.com>
Subject: Re: svn commit: r1300690 - in
 /directory/apacheds/branches/apacheds-txns:
 core-api/src/main/java/org/apache/directory/server/core/api/log/
 core-api/src/main/java/org/apache/directory/server/core/api/txn/
 core-api/src/main/java/org/apache/directory/se
From: Selcuk AYA <ayaselcuk@gmail.com>
To: Apache Directory Developers List <dev@directory.apache.org>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

HI All,
Sorry for the earlier email. I think I owe some explaination on my
part. The reason for my request is purely technical, does not aim at
oss spirit or any other spirit for that matter:
* There are quite a number of files the txn branch is touching.
* There is no file ownership or review process.

combined with the timing limitation, it becomes hard for me to track
all changes and cleanup if necessary. When I am doing my changes and
need to change some existing stuff, I usually try to find the guy who
wrote the code and get an ack from him and this usually helps a lot
because even things that look stupid might have some reason to be
there. Please do the same while changing the txn branch.If this
process is followed, we wont have to discuss spirit hurting through
reverting code.

thanks
Selcuk

On Wed, Mar 14, 2012 at 2:48 PM, Alex Karasulu <akarasulu@apache.org> wrote=
:
>
>
> On Wed, Mar 14, 2012 at 10:41 PM, Kiran Ayyagari <kayyagari@apache.org>
> wrote:
>>
>> Selcuk,
>>
>> =A0 =A0I have seen you asking several times on this list for reverting
>> commits,
>> =A0 =A0this seems to be a bit derogatory in OSS spirit and team work.
>
>
> It can certainly be=A0misinterpreted=A0this way. I think we just need mor=
e
> communication about why one may need a revert.
>
> Revert requests are OK. There's nothing wrong with that and any committer
> can veto a change but they just need to provide reasons. I think we just
> need to help people understand this.
>
> Selcuk I'm sure meant no harm and can provide more reasoning.
>
>>
>> =A0 =A0Go ahead and make your changes on top of these if you wish to app=
ly
>> =A0 =A0your fix, commits need not be reverted for this.
>>
>
> That's also another option but let's just communicate about whatever
> difficulties or problems a commit might introduce. If someone is having
> problems as a result of commits let's get those reasons out there on the
> list.
>
> I have a feeling the code is starting to move before some of the work can=
 be
> finished on it and that might produce discomfort. But we're not mind read=
ers
> so we need to communicate this.
>
>>
>> On Thu, Mar 15, 2012 at 1:39 AM, Selcuk AYA <ayaselcuk@gmail.com> wrote:
>> > please revert this commit.
>> >
>
>
> --
> Best Regards,
> -- Alex
>