[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

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

From: Branko Čibej <brane_at_wandisco.com>
Date: Sun, 18 Aug 2013 14:27:32 +0200

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
Received on 2013-08-18 14:28:08 CEST

This is an archived mail posted to the Subversion Dev mailing list.