Re: svn commit: r1462054 - in /subversion/branches/verify-at-commit/subversion: include/svn_fs.h libsvn_fs/fs-loader.c libsvn_fs/fs-loader.h

Stefan Sperling <stsp_at_elego.de> writes:

> If we decide that verifying transactions before commit is a good
> thing, I'd much rather see a new 'svnlook verify' command that can
> be run from the pre-commit hook, instead of this implementation.

A separate process could be a performance hit since it would not share
the FSFS cache used by the commit process. On the other hand a separate
process might be an advantage if there is a caching bug.

-- 
Certified & Supported Apache Subversion Downloads:
http://www.wandisco.com/subversion/download