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:
=====================