Re: [PATCH] svn_fs_text_changed

On Wed, 2003-11-26 at 00:37, C. Michael Pilato wrote:
> > With patch: average is 4 seconds; without patch: average is 6 seconds.
> Hey, that's good news. Now, revert your change, and do those same
> tests after teaching your window_handler to notice no-op textdeltas.
> Or, if you don't have time, I'd be willing to do this test comparison
> myself -- just send me your recipe.

It's quite simple: make a local import of svn tree at rev6000 or so,
create a local branch, update the local import to trunk, do a merge to
the local branch. Now diff the local import and the branch.

btw, a local ra_dav test results in 15secs vs 18secs, user time 1.15 vs
1.8, system time 0.6 vs 1.0. Supposed the noop window fixed in client
which saves all the user/sys time diff, it's still a ~2-second win.

> > But this is the case that my intention. I didn't do other tests to see
> > if it degrades normal cases.
>
> Well, yeah, those are the cases I had in mind. I rather figured you
> weren't out to degrade your own use-case. :-)

But if people thinks it's overall a good thing, I'll polish my patch for
the 2nd round.

Cheers,
CLK

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Nov 25 18:27:45 2003