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

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.
>

From a quick review of the patch, I am not seeing how we are delaying the
client's editor drive to set the property until we have received the file
contents. As I read it, we would set the properties on each file as we
walk the REPORT response.

As I understand it, we have repeatedly stated here on list that we are not
supposed to tell the editor (esp. Ev2) about any of the properties until we
have the file contents too because then the editor would have too many
handles open and kill us with resource exhaustion. -- justin
Received on 2012-06-10 09:29:08 CEST