<div dir="ltr">Hello,<br><div><br>On Wed, Mar 9, 2016 at 4:24 PM, Daniel Shahaf <span dir="ltr"><<a href="mailto:danielsh@apache.org" target="_blank">danielsh@apache.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Concerning the 'The Three Meanings of "Lock"' box:<br>
.<br>
    <a href="http://svnbook.red-bean.com/nightly/en/svn.advanced.locking.html#svn.advanced.locking.meanings" rel="noreferrer" target="_blank">http://svnbook.red-bean.com/nightly/en/svn.advanced.locking.html#svn.advanced.locking.meanings</a><br>
.<br>
Should that box mention svnsync locks as well?<br>
<br>
(These are the svn_ra__get_operational_lock() locks that svnsync takes<br>
using the svn:sync-lock revprop, exposed in the UI through the<br>
--disable-locks/--steal-lock options to 'svnsync sync'.  Note 'svnrdump<br>
load' also uses this kind of operational lock.)<br>
</blockquote></div><br>I doubt that the revprop-based lock `svnsync` or `svnrdump` places is worth mentioning in the  'The Three Meanings of "Lock"' section.<br><br></div><div class="gmail_extra">`svnsync` and `svnrdump` place `svn:sync-lock` and `svn:rdump-lock` revprops to revision 0 and it's an implementation detail, not a real concept. It's just how it works in Subversion 1.9 and older versions. Thinking about it, this could easily change in the future, should `svnsync` or `svnrdump` stop placing such kind of locks.<br><br></div><div class="gmail_extra">IMO, the section describes the 3 basic locks like administrative and database and it should not contain maintenance revprop-based locks at all. However, I think that the "Third, there are database locks, used internally by the Berkeley DB..." part could be updated since Berkeley DB is deprecated starting with SVN 1.9".<br></div><div class="gmail_extra"><br>--<br><div><font color="#888888">With best regards,<br>
Pavel Lyalyakin<br>
VisualSVN Team</font></div>
</div></div></div>