Author: philip
Date: Tue Mar 18 12:57:22 2014
New Revision: 1578853

URL: http://svn.apache.org/r1578853
Log:
Make svnserve recognise when the client does not want inherited
properties.  This fixes a performance problem with 1.8 servers
sending too much data.

Reported by: "Matthews, David" <david.matthews{_AT_}metoffice.gov.uk>

* subversion/svnserve/serve.c
  (get_file, get_dir): Properly handle the optional wants_inherited_props
   flag to avoid sending inherited properties when not requested.

Modified:
    subversion/trunk/subversion/svnserve/serve.c

Modified: subversion/trunk/subversion/svnserve/serve.c
URL: 
http://svn.apache.org/viewvc/subversion/trunk/subversion/svnserve/serve.c?rev=1578853&r1=1578852&r2=1578853&view=diff
==============================================================================
--- subversion/trunk/subversion/svnserve/serve.c (original)
+++ subversion/trunk/subversion/svnserve/serve.c Tue Mar 18 12:57:22 2014
@@ -1536,6 +1536,9 @@ static svn_error_t *get_file(svn_ra_svn_
                                   &want_props, &want_contents,
                                   &wants_inherited_props));
 
+  if (wants_inherited_props == SVN_RA_SVN_UNSPECIFIED_NUMBER)
+    wants_inherited_props = FALSE;
+
   full_path = svn_fspath__join(b->repository->fs_path->data,
                                svn_relpath_canonicalize(path, pool), pool);
 
@@ -1650,6 +1653,9 @@ static svn_error_t *get_dir(svn_ra_svn_c
                                   &dirent_fields_list,
                                   &wants_inherited_props));
 
+  if (wants_inherited_props == SVN_RA_SVN_UNSPECIFIED_NUMBER)
+    wants_inherited_props = FALSE;
+
   if (! dirent_fields_list)
     {
       dirent_fields = SVN_DIRENT_ALL;


Reply via email to