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/09/21 18:06:16 UTC

svn commit: r1704343 - /subversion/branches/ra_serf-stream-commit/BRANCH-README

Author: kotkov
Date: Mon Sep 21 16:06:11 2015
New Revision: 1704343

URL: http://svn.apache.org/viewvc?rev=1704343&view=rev
Log:
On the ra_serf-stream-commit branch: Update BRANCH-README.

* BRANCH-README
  (STATUS): Explain current status of the branch in this new section.

Modified:
    subversion/branches/ra_serf-stream-commit/BRANCH-README

Modified: subversion/branches/ra_serf-stream-commit/BRANCH-README
URL: http://svn.apache.org/viewvc/subversion/branches/ra_serf-stream-commit/BRANCH-README?rev=1704343&r1=1704342&r2=1704343&view=diff
==============================================================================
--- subversion/branches/ra_serf-stream-commit/BRANCH-README (original)
+++ subversion/branches/ra_serf-stream-commit/BRANCH-README Mon Sep 21 16:06:11 2015
@@ -7,3 +7,21 @@ associated with writing these files, apa
 requires a certain amount of disk space.
 
 The branch should be maintained with catch-up merges from trunk.
+
+STATUS
+======
+
+Quick implementation showed that there is no visible overhead associated with
+the temporary files.  In my environment, the streaming approach behaves almost
+exactly the same, but implementing it adds a *lot of complexity*.  Perhaps
+doing so could spare us a few seconds when committing large files â e.g.,
+.iso â but overall, the complexity is not worth the benefit.
+
+A couple of observations that happened while developing this branch resulted
+in the r1704317 change [1] (Use compressed svndiff1 format for deltas when
+committing changes over http:// and https://) committed to trunk.
+
+There are no plans on continuing this branch.  It's left in its current state,
+in case it could get useful in the future.
+
+[1] https://svn.apache.org/r1704317