Author: hwright
Date: Mon May 16 09:51:39 2011
New Revision: 1103665

URL: http://svn.apache.org/viewvc?rev=1103665&view=rev
Log:
Fix a user-reported performance issue where we don't properly retrieve the
cached manifest offset value in FSFS.  See
http://svn.haxx.se/dev/archive-2011-05/0428.shtml

Even though on trunk we are currently using apr_int64_t, danielsh tells me
that svn_revnum_t is a "more proper" type.

Patch by: Roderich Schupp <[email protected]>

* subversion/libsvn_fs_fs/fs_fs.c
  (get_packed_offset): Use the proper type for the shard.

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

Modified: subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c
URL: 
http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c?rev=1103665&r1=1103664&r2=1103665&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c (original)
+++ subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c Mon May 16 09:51:39 2011
@@ -1955,7 +1955,7 @@ get_packed_offset(apr_off_t *rev_offset,
   fs_fs_data_t *ffd = fs->fsap_data;
   svn_stream_t *manifest_stream;
   svn_boolean_t is_cached;
-  apr_int64_t shard;
+  svn_revnum_t shard;
   apr_int64_t shard_pos;
   apr_array_header_t *manifest;
   apr_pool_t *iterpool;


Reply via email to