"David Glasser" <glasser@davidglasser.net> writes:
> Here's a simple patch that makes the test I added in r27575 pass. The
> whole test suite passes with this patch applied, but since this bug
> involves error behavior, the test suite might not actually cover it
> well. I would like review before I commit.
>
> (An alternate implementation, which would still allow *some* in-memory
> logs to be run during cleanup, would be to have a boolean
> "logs_ok_to_run" flag on the directory baton, which is cleared every
> time the accumulator is appended to and set only when the accumulator
> is definitely runnable.)
>
> --dave
>
> [[[
> Fix wc corruption caused by flushing potentially-incomplete logs
> during baton cleanup on error, by just not flushing logs during baton
> cleanup. Makes the new update test #42 pass.
>
> * subversion/libsvn_wc/update_editor.c
> (cleanup_dir_baton): Don't call flush_log.
>
> * subversion/tests/cmdline/update_tests.py
> (test_list): eof_in_interactive_conflict_resolver now passes.
> ]]]
It's certainly a serious mistake to run incomplete log files (unless
the implementation has had a major rework, I haven't really been
following the development recently).
Your alternate implementation sounds much better: as I understand it
your proposed patch means that if a large update is interrupted there
could be lots of log "files" in memory corresponding to megabytes of
downloaded data and effectively all this data would be lost.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Nov 6 01:42:30 2007