Hi,

On Thu, Feb 28, 2008 at 11:47 AM,  <[EMAIL PROTECTED]> wrote:
> Author: thomasm
>  Date: Thu Feb 28 01:47:51 2008
>  New Revision: 631905
>
>  URL: http://svn.apache.org/viewvc?rev=631905&view=rev
>  Log:
>  JCR-1414 Data store garbage collection: inUse not correctly synchronized
> [...]
>  Modified: 
> jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/persistence/bundle/BundleDbPersistenceManager.java
>  URL: 
> http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/persistence/bundle/BundleDbPersistenceManager.java?rev=631905&r1=631904&r2=631905&view=diff
>  
> ==============================================================================
>  --- 
> jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/persistence/bundle/BundleDbPersistenceManager.java
>  (original)
>  +++ 
> jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/persistence/bundle/BundleDbPersistenceManager.java
>  Thu Feb 28 01:47:51 2008
>  @@ -931,7 +931,7 @@
>                  // see also bundleSelectAllIdsFrom SQL statement
>                  maxCount += 10;
>              }
>  -            Statement stmt = connectionManager.executeStmt(sql, keys, 
> false, maxCount + 10);
>  +            Statement stmt = connectionManager.executeStmt(sql, keys, 
> false, maxCount);
>              rs = stmt.getResultSet();
>              ArrayList result = new ArrayList();
>              while ((maxCount == 0 || result.size() < maxCount) && rs.next()) 
> {

I have trouble understanding this bit. Why is this change needed?

BR,

Jukka Zitting

Reply via email to