You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by st...@apache.org on 2015/10/11 19:15:58 UTC

svn commit: r1708004 - /subversion/trunk/subversion/libsvn_fs_fs/verify.c

Author: stefan2
Date: Sun Oct 11 17:15:58 2015
New Revision: 1708004

URL: http://svn.apache.org/viewvc?rev=1708004&view=rev
Log:
Speed up verification of FSFS repositories with packed revprops.

* subversion/libsvn_fs_fs/verify.c
  (verify_revprops): Invalidate the revprop cache only once and then
                     use it as read-ahead buffer.

Modified:
    subversion/trunk/subversion/libsvn_fs_fs/verify.c

Modified: subversion/trunk/subversion/libsvn_fs_fs/verify.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_fs_fs/verify.c?rev=1708004&r1=1708003&r2=1708004&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_fs_fs/verify.c (original)
+++ subversion/trunk/subversion/libsvn_fs_fs/verify.c Sun Oct 11 17:15:58 2015
@@ -30,6 +30,7 @@
 
 #include "cached_data.h"
 #include "rep-cache.h"
+#include "revprops.h"
 #include "util.h"
 #include "index.h"
 
@@ -721,6 +722,10 @@ verify_revprops(svn_fs_t *fs,
   svn_revnum_t revision;
   apr_pool_t *iterpool = svn_pool_create(pool);
 
+  /* Invalidate the revprop cache once.
+   * Use the cache inside the loop to speed up packed revprop access. */
+  svn_fs_fs__reset_revprop_cache(fs);
+
   for (revision = start; revision < end; ++revision)
     {
       svn_string_t *date;
@@ -731,7 +736,7 @@ verify_revprops(svn_fs_t *fs,
       /* Access the svn:date revprop.
        * This implies parsing all revprops for that revision. */
       SVN_ERR(svn_fs_fs__revision_prop(&date, fs, revision,
-                                       SVN_PROP_REVISION_DATE, TRUE,
+                                       SVN_PROP_REVISION_DATE, FALSE,
                                        iterpool, iterpool));
 
       /* The time stamp is the only revprop that, if given, needs to