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);
}