How about a comment in the code? The log message is not the proper location to document why a query is constructed in a certain way. On May 19, 2012 7:46 PM, <[email protected]> wrote:
> Author: rhuijben > Date: Sat May 19 23:45:56 2012 > New Revision: 1340592 > > URL: http://svn.apache.org/viewvc?rev=1340592&view=rev > Log: > * subversion/libsvn_wc/wc-queries.sql > (STMT_HAS_SERVER_EXCLUDED_NODES): Only select descendants of the node as > we > only call it on existing nodes. This makes sqlite handle the right index > and shaves about 60% of the time to delete individual 400 files from my > working copy. > > Modified: > subversion/trunk/subversion/libsvn_wc/wc-queries.sql > > Modified: subversion/trunk/subversion/libsvn_wc/wc-queries.sql > URL: > http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_wc/wc-queries.sql?rev=1340592&r1=1340591&r2=1340592&view=diff > > ============================================================================== > --- subversion/trunk/subversion/libsvn_wc/wc-queries.sql (original) > +++ subversion/trunk/subversion/libsvn_wc/wc-queries.sql Sat May 19 > 23:45:56 2012 > @@ -884,9 +884,8 @@ LIMIT 1 > -- STMT_HAS_SERVER_EXCLUDED_NODES > SELECT local_relpath FROM nodes > WHERE wc_id = ?1 > - AND (?2 = '' > - OR local_relpath = ?2 > - OR IS_STRICT_DESCENDANT_OF(local_relpath, ?2)) > + AND (parent_relpath = ?2 > + OR IS_STRICT_DESCENDANT_OF(parent_relpath, ?2)) > AND op_depth = 0 AND presence = 'absent' > LIMIT 1 > > > >
