Re: svn commit: r1348131 - /subversion/trunk/subversion/libsvn_ra_serf/update.c

On Jun 10, 2012 3:20 AM, "Justin Erenkrantz" <justin_at_erenkrantz.com> wrote:
>
> On Friday, June 8, 2012, C. Michael Pilato wrote:
>>
>> On 06/08/2012 03:15 PM, Ivan Zhakov wrote:
>> > Wow, but performance will be much more interesting if we inline
>> > properties to REPORT response. Mike, could you please share your
>> > incomplete patch for including properties to REPORT response?
>>
>> Sure. Attached.
>
>
> I would still prefer the PROPFIND Depth 1 strategy as that won't be
dependent upon a server upgrade.
>
> At the least, this patch will give us a ballpark figure to shoot for in
reducing the per-file PROPFINDs...maybe we ought to implement both
techniques, but I think Depth 1 will bring us pretty close and not require
server changes...I expect the delta in perf between these two won't be
huge...we can always do a bake-off! -- justin

Right. I'd like to see Depth:1, as it can work against all servers (even
1.0).

Then putting props in the 1.8 server response would help a little bit more.

Cheers,
-g
Received on 2012-06-10 14:50:15 CEST