svn commit: r1708000 - /subversion/trunk/subversion/libsvn_repos/dump.c

[st...@apache.org]

Author: stefan2
Date: Sun Oct 11 16:57:55 2015
New Revision: 1708000

URL: http://svn.apache.org/viewvc?rev=1708000&view=rev
Log:
Speed up revprop access in svn_repos_dump_fs4() by using the latest FS API.

* subversion/libsvn_repos/dump.c
  (write_revision_record): Read revprops from the FS cache if we can. 
  (svn_repos_dump_fs4): Refresh the revprops only once, at the start of the
                        operation.  This is the only public function that
                        calls the above.  Therefore, we still fulfill the
                        visibility guarantees for revprop changes.

Modified:
    subversion/trunk/subversion/libsvn_repos/dump.c

Modified: subversion/trunk/subversion/libsvn_repos/dump.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_repos/dump.c?rev=1708000&r1=1707999&r2=1708000&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_repos/dump.c (original)
+++ subversion/trunk/subversion/libsvn_repos/dump.c Sun Oct 11 16:57:55 2015
@@ -1934,7 +1934,7 @@ write_revision_record(svn_stream_t *stre
 
   if (include_revprops)
     {
-      SVN_ERR(svn_fs_revision_proplist(&props, fs, rev, pool));
+      SVN_ERR(svn_fs_revision_proplist2(&props, fs, rev, FALSE, pool, pool));
 
       /* Run revision date properties through the time conversion to
         canonicalize them. */
@@ -1991,6 +1991,10 @@ svn_repos_dump_fs4(svn_repos_t *repos,
   svn_boolean_t found_old_mergeinfo = FALSE;
   svn_repos_notify_t *notify;
 
+  /* Make sure we catch up on the latest revprop changes.  This is the only
+   * time we will refresh the revprop data in this query. */
+  SVN_ERR(svn_fs_refresh_revision_props(fs, pool));
+
   /* Determine the current youngest revision of the filesystem. */
   SVN_ERR(svn_fs_youngest_rev(&youngest, fs, pool));