RE: svn commit: r1353676 - in /subversion/trunk/subversion: include/private/svn_wc_private.h include/svn_ra.h libsvn_client/ra.c libsvn_ra_serf/update.c libsvn_wc/adm_ops.c

> -----Original Message-----
> From: cmpilato_at_apache.org [mailto:cmpilato_at_apache.org]
> Sent: maandag 25 juni 2012 19:32
> To: commits_at_subversion.apache.org
> Subject: svn commit: r1353676 - in /subversion/trunk/subversion:
> include/private/svn_wc_private.h include/svn_ra.h libsvn_client/ra.c
> libsvn_ra_serf/update.c libsvn_wc/adm_ops.c
>
> Author: cmpilato
> Date: Mon Jun 25 17:32:26 2012
> New Revision: 1353676
>
> URL: http://svn.apache.org/viewvc?rev=1353676&view=rev
> Log:
> Teach the get_wc_content() RA callback and supporting code to handle
> MD5 checksums as content indexes, too. This *should* result in the
> benefits of this enhancement extending to users of all 1.8 clients
> irrespective of the server's pedigree, as MD5s have been in the server
> streams since ... like ... forever.

If we want to enable this code in 1.8 we should add an index on the md5_checksum in the pristine table with the format bump for 1.8.

Currently the md5 lookup performs a table scan (but is only used from the deprecated libsvn_wc commit logic). For that 1.5 million file working copy reported today on users_at_s.a.o, this code might be much slower than just downloading the files again without an index.

        Bert
Received on 2012-06-25 20:06:01 CEST