Re: svn commit: r1515088 - in /subversion/branches/log-addressing/subversion: include/private/svn_subr_private.h include/svn_checksum.h libsvn_subr/checksum.c libsvn_subr/fnv1a.c libsvn_subr/fnv1a.h tests/libsvn_subr/checksum-test.c

On 18.08.2013 14:25, Branko Čibej wrote:
> On 18.08.2013 14:15, stefan2_at_apache.org wrote:
>> Author: stefan2
>> Date: Sun Aug 18 12:15:01 2013
>> New Revision: 1515088
>>
>> URL: http://svn.apache.org/r1515088
>> Log:
>> On the log-addressing branch: For low-overhead checksumming,
>> add standard FNV-1a and a faster modified version of that to
>> the list of checksum kinds supported with svn_checksum_t.
>>
>> We will use this new checksum to secure parts of FSFS (and
>> later FSX) that are not directly covered by MD5/SHA1 content
>> checksums. That will help to localize corruptions much quicker
>> and more accurately but it will not eliminate the need to run
>> a full content verification.
> If you're using this for detecting corruption, rather than key
> distribution, why not instead use a 64-bit or even 32-bit CRC? It should
> be much faster than any kind of multiply-with-prime hash.

Or you could even use the Adler-32 implementation that we already use in
the xdelta code.

-- Brane

-- 
Branko Čibej | Director of Subversion
WANdisco // Non-Stop Data
e. brane_at_wandisco.com