Julian Foad <julianf...@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*