Re: svn commit: r1680819 - /subversion/trunk/subversion/libsvn_fs_fs/revprops.c

Julian Foad <julianfoad_at_btopenworld.com> writes:

> Evgeny Kotkov wrote:
>> Philip, Julian, thank you for giving this patch a look.
>>
>> As I said in my previous e-mails, I think that the committed fix is a better
>> choice here, as opposed to attempts to achieve the same behavior using the
>> stream API. [...]
>
> That sounds good to me -- leaving the code using file APIs.

It only fixes writing revprops during commit, it still leaves the
problem that the pack operation doesn't flush. We need to add flush
when writing the revision pack file, the packed indices, the revprop
pack file and the revprop manifest file. That means replacing more, if
not all, of the stream code in revprops.c.

-- 
Philip Martin | Subversion Committer
WANdisco // *Non-Stop Data*