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

On Jun 10, 2012 3:29 AM, "Justin Erenkrantz" <justin_at_erenkrantz.com> wrote:
>
> On Friday, June 8, 2012, C. Michael Pilato wrote:
>>...
>> 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

Correct. With Ev2, the props need to be stashed until the content arrives.

Cheers,
-g
Received on 2012-06-10 14:54:00 CEST