Re: Performance branch ready for review

2010-08-23 Thread Johan Corveleyn
On Tue, Aug 24, 2010 at 12:22 AM, Stefan Fuhrmann wrote: > Daniel Shahaf wrote: >> >> Stefan, you did mention "Patch by" for Johan's patches which you >> committed, did you intend to mention "Found by" or "Suggested by" for >> the other two (quoted below)? >> >> >> http://subversion.apache.org/doc

Re: Performance branch ready for review

2010-08-23 Thread Stefan Fuhrmann
Daniel Shahaf wrote: Stefan, you did mention "Patch by" for Johan's patches which you committed, did you intend to mention "Found by" or "Suggested by" for the other two (quoted below)? http://subversion.apache.org/docs/community-guide/conventions.html#crediting Thanks, Daniel Oh, I just w

Re: Performance branch ready for review

2010-08-23 Thread Johan Corveleyn
On Sun, Aug 22, 2010 at 2:56 PM, Stefan Fuhrmann wrote: > Johan Corveleyn wrote: >> >> On Wed, Aug 18, 2010 at 9:14 PM, Stefan Fuhrmann >> wrote: >> >>> >>> Hi @all, >>> >>> I just finished my porting work; the performance branch >>> is now fully synchronized with my prototype code. >>> From my p

Re: Performance branch ready for review

2010-08-22 Thread Daniel Shahaf
Stefan, you did mention "Patch by" for Johan's patches which you committed, did you intend to mention "Found by" or "Suggested by" for the other two (quoted below)? http://subversion.apache.org/docs/community-guide/conventions.html#crediting Thanks, Daniel Stefan Fuhrmann wrote on Sun, Aug 22,

Re: Performance branch ready for review

2010-08-22 Thread Stefan Fuhrmann
Johan Corveleyn wrote: On Wed, Aug 18, 2010 at 9:14 PM, Stefan Fuhrmann wrote: Hi @all, I just finished my porting work; the performance branch is now fully synchronized with my prototype code. From my point of view, review can start now. According to my measurements, the code is now faste

RE: Performance branch ready for review

2010-08-18 Thread Bert Huijben
> -Original Message- > From: Johan Corveleyn [mailto:jcor...@gmail.com] > Sent: woensdag 18 augustus 2010 15:59 > To: Stefan Fuhrmann > Cc: Subversion Development > Subject: Re: Performance branch ready for review > > On Wed, Aug 18, 2010 at 9:14 PM, Stefan F

Re: Performance branch ready for review

2010-08-18 Thread Johan Corveleyn
On Wed, Aug 18, 2010 at 9:14 PM, Stefan Fuhrmann wrote: > Hi @all, > > I just finished my porting work; the performance branch > is now fully synchronized with my prototype code. > From my point of view, review can start now. > > According to my measurements, the code is now faster > than the orig