svn commit: r983803 - /subversion/branches/atomic-revprop/BRANCH-README

[da...@apache.org]

Author: danielsh
Date: Mon Aug  9 20:13:33 2010
New Revision: 983803

URL: http://svn.apache.org/viewvc?rev=983803&view=rev
Log:
On the 'atomic-revprop' branch:

* BRANCH-README (ra_dav):  Add current plan, after discussion with cmpilato.

Modified:
    subversion/branches/atomic-revprop/BRANCH-README

Modified: subversion/branches/atomic-revprop/BRANCH-README
URL: http://svn.apache.org/viewvc/subversion/branches/atomic-revprop/BRANCH-README?rev=983803&r1=983802&r2=983803&view=diff
==============================================================================
--- subversion/branches/atomic-revprop/BRANCH-README (original)
+++ subversion/branches/atomic-revprop/BRANCH-README Mon Aug  9 20:13:33 2010
@@ -23,14 +23,37 @@ Planned work
   - [DONE] ra_svn: Add a 'change-rev-prop2' verb.
   - [DONE] ra_dav: Extend PROPPATCH: see notes/http-and-webdav/webdav-protocol
   - [DONE] unit test: prop_tests.py 34: atomic_over_ra()
+  - TODO: ensure consistent error code (replacing SVN_ERR_BAD_PROPERTY_VALUE)
+  - TODO: use "409 Conflict" DAV response (within the 207 Multi-Status response)
 
 * [DONE] client support: svn_client_revprop_set2()
 
-* svnsync:
+* TODO: svnsync:
   - use the new RA API to resolve the race condition in locking.
   - upgrade svn_ra_change_rev_prop() callers
 
 
+Dependency: have ra_dav provide err->apr_err over the wire
+==========================================================
+
+The svnsync work requires having the API return a known error code if the
+propchange failed /due to the provided expectation *old_value_p not having
+been matched/.
+
+Currently, ra_local and ra_svn have the original error code in the chain, but
+mod_dav_svn only has a generic SVN_ERR_RA_DAV_* error code (despite having the
+correct error message).
+
+* TODO: verify ra_local and ra_svn claims above
+* TODO: extend ra_dav to transmit more error information over the wire
+  plan1:
+    - nest a "<S:errcode>42</S:errcode>" tag inside the <D:responsedescription> tag.
+      (mod_dav doesn't XML-encode dav_error->desc)
+    - extend ra_neon/ra_serf's start_207()/start_207_element() parsers
+  plan2:
+    (?)
+
+
 Potential work
 ==============