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))