Re: [Issue 1240] Changed - svn_subst_translate_stream() should read bigger chunks

On Fri, Apr 11, 2003 at 08:40:33AM -0500, Karl Fogel wrote:
> "Tim Kemp" <subversion@kempy.co.uk> writes:
> > Hmmmm, 10% of checkout time is spent in svn_subst_copy_and_translate.
> > With 10 million calls all the way down to apr_file_read and
> > apr_file_write.
>
> That's a very bizarre result.
>
> Ben's export editor, before he added the translation calls, was 4
> times faster than a regular checkout. When he added translation, it
> went back to being about the same amount of time as a checkout. (I
> will check this to make sure, but I think adding translation boils
> down to svn_subst_copy_and_translate() being called for each file.)
>
> And yet, you're saying that only 10% of checkout time is spent in
> svn_subst_copy_and_translate().
>
> At least one of these sets of numbers is misleading... :-)

Has Ben re-verified his numbers? Let's make sure that the numbers
we're all scurrying about for, are actually valid.

Regardless... 10% of checkout time is significant, thus is a fine
candidate for examination.

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org