You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by gs...@apache.org on 2012/05/07 04:53:28 UTC

svn commit: r1334866 - /subversion/trunk/subversion/libsvn_ra_serf/locks.c

Author: gstein
Date: Mon May  7 02:53:28 2012
New Revision: 1334866

URL: http://svn.apache.org/viewvc?rev=1334866&view=rev
Log:
Leave a note for the future.

* subversion/libsvn_ra_serf/locks.c:
  (svn_ra_serf__lock): note that we could pipeline these requests

Modified:
    subversion/trunk/subversion/libsvn_ra_serf/locks.c

Modified: subversion/trunk/subversion/libsvn_ra_serf/locks.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_ra_serf/locks.c?rev=1334866&r1=1334865&r2=1334866&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_ra_serf/locks.c (original)
+++ subversion/trunk/subversion/libsvn_ra_serf/locks.c Mon May  7 02:53:28 2012
@@ -562,7 +562,10 @@ svn_ra_serf__lock(svn_ra_session_t *ra_s
   iterpool = svn_pool_create(scratch_pool);
 
   /* ### TODO for issue 2263: Send all the locks over the wire at once.  This
-     loop is just a temporary shim. */
+     ### loop is just a temporary shim.
+     ### an alternative, which is backwards-compat with all servers is to
+     ### pipeline these requests. ie. stop using run_wait/run_one.  */
+
   for (hi = apr_hash_first(scratch_pool, path_revs);
        hi;
        hi = apr_hash_next(hi))