You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by gt...@apache.org on 2012/10/16 14:54:09 UTC

svn commit: r1398770 - /activemq/trunk/kahadb/src/main/java/org/apache/kahadb/index/BTreeNode.java

Author: gtully
Date: Tue Oct 16 12:54:09 2012
New Revision: 1398770

URL: http://svn.apache.org/viewvc?rev=1398770&view=rev
Log:
https://issues.apache.org/jira/browse/AMQ-4094 - revert inadvertent mod/regression to BTreeNode, multiple iterations of the test verify it is unnecessary and seems to be the root cause of some instability

Modified:
    activemq/trunk/kahadb/src/main/java/org/apache/kahadb/index/BTreeNode.java

Modified: activemq/trunk/kahadb/src/main/java/org/apache/kahadb/index/BTreeNode.java
URL: http://svn.apache.org/viewvc/activemq/trunk/kahadb/src/main/java/org/apache/kahadb/index/BTreeNode.java?rev=1398770&r1=1398769&r2=1398770&view=diff
==============================================================================
--- activemq/trunk/kahadb/src/main/java/org/apache/kahadb/index/BTreeNode.java (original)
+++ activemq/trunk/kahadb/src/main/java/org/apache/kahadb/index/BTreeNode.java Tue Oct 16 12:54:09 2012
@@ -332,9 +332,10 @@ public final class BTreeNode<Key,Value> 
                         keys = child.keys;
                         children = child.children;
                         values = child.values;
+                        // free up the page..
+                        tx.free(child.getPage());
                     }
-                    // free up the page..
-                    tx.free(child.getPage());
+                    
                 }
                 index.storeNode(tx, this, true);
             }