There was in fact a test break. I fixed it and will include it in my next patch.
In issue 2151 (http://subversion.tigris.org/issues/show_bug.cgi?id=2151):
"The URIs listed in gather_propsets() are returned in
the OPTIONS response. Since we always(?) issue an OPTIONS request, then we
extract them at that point, recording the presence in the session state."
Unfortunatly, from what I can see int the code we only issue an
OPTIONS request on commit. Should we always do it? I will try to add
it and see if there are any issues that pop up.
On 8/26/05, Jean-Marc Godbout <jmgodbout@gmail.com> wrote:
> Oh, I'm sorry, I just realised I had not run all the tests. Please
> don't waste your time looking at this patch until I make sure it
> doesn't break tests.
>
> On 8/26/05, Jean-Marc Godbout <jmgodbout@gmail.com> wrote:
> > I have a simple patch that trims down time by about half in my tests.
> >
> > It's NOT complete, here's what works:
> >
> > 1. Patched client + Patched server == fast
> > 2. Patched client + Unpatched server == broken
> > 3. Unpatched client + Patched server == same speed as before
> > 4. Dav client + Patched server == same speed as before
> >
> > I need to fix #2 before considering it a real patch.
> >
> > Essentially this implements some of the suggestions in 2151 - starting
> > with Ben's patch. Note that I don't think it's wise to remove DeltaV
> > props from an allprop PROPFIND as this would break unpatched clients
> > trying to access patched servers. We can discuss this though.
> >
> > On 8/25/05, Carsten Koch <Carsten.Koch@icem.com> wrote:
> > > Jean-Marc Godbout wrote:
> > > > For Ben's suggestion of using mod_deflate, I think it could be useful
> > > > in your case. I just ran a test on a real system and a sample svn list
> > > > went from 60k to 16k.
> > >
> > > We have mod_deflate active now on our svn server and the small test case
> > > (svn list -R on a directory tree with only 25 entries) goes down from
> > > 8 seconds to 3 seconds.
> > > Not the facto 100 improvement I am looking for, but nevertheless more
> > > than twice as fast.
> > >
> > > Thanks to Ben and you for the suggestion!
> > >
> > > Carsten.
> > >
> > > ---------------------------------------------------------------------
> > > To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
> > > For additional commands, e-mail: dev-help@subversion.tigris.org
> > >
> > >
> > >
> >
> >
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
> For additional commands, e-mail: dev-help@subversion.tigris.org
>
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Aug 27 02:20:41 2005