You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by ko...@apache.org on 2015/12/06 13:57:16 UTC

svn commit: r1718172 - /subversion/branches/1.9.x/STATUS

Author: kotkov
Date: Sun Dec  6 12:57:16 2015
New Revision: 1718172

URL: http://svn.apache.org/viewvc?rev=1718172&view=rev
Log:
* STATUS: Nominate the r1718167 changeset.

Modified:
    subversion/branches/1.9.x/STATUS

Modified: subversion/branches/1.9.x/STATUS
URL: http://svn.apache.org/viewvc/subversion/branches/1.9.x/STATUS?rev=1718172&r1=1718171&r2=1718172&view=diff
==============================================================================
--- subversion/branches/1.9.x/STATUS (original)
+++ subversion/branches/1.9.x/STATUS Sun Dec  6 12:57:16 2015
@@ -110,6 +110,20 @@ Candidate changes:
    Votes:
      +1: rhuijben
 
+ * r1718167
+   Disable zero-copy code path for ra_local update reporters.
+   Justification:
+     Using zero-copy code path within ra_local is not safe.
+     .
+     The known and documented limitation of svn_repos_begin_report3()
+     is that with zero-copy enabled, the delta editor callbacks cannot
+     access FSFS or use Subversion caches directly.  Within ra_local,
+     we cannot be sure that arbitrary callers of our RA API, such as
+     svn_ra_do_update3(), are aware of this limitation and pass-in
+     the delta editor that doesn't access FSFS or caches.
+   Votes:
+     +1: kotkov
+
 Veto-blocked changes:
 =====================