JBoss Cache SVN: r4722 - core/trunk/src/test/java/org/jboss/cache/statetransfer.
by jbosscache-commits@lists.jboss.org
Author: manik.surtani(a)jboss.com
Date: 2007-11-01 13:04:24 -0400 (Thu, 01 Nov 2007)
New Revision: 4722
Modified:
core/trunk/src/test/java/org/jboss/cache/statetransfer/StateTransferUnderLoadTest.java
Log:
Added a reason as to why the test is disabled.
Modified: core/trunk/src/test/java/org/jboss/cache/statetransfer/StateTransferUnderLoadTest.java
===================================================================
--- core/trunk/src/test/java/org/jboss/cache/statetransfer/StateTransferUnderLoadTest.java 2007-11-01 12:15:10 UTC (rev 4721)
+++ core/trunk/src/test/java/org/jboss/cache/statetransfer/StateTransferUnderLoadTest.java 2007-11-01 17:04:24 UTC (rev 4722)
@@ -24,7 +24,7 @@
* @author Bela Ban
* @version $Id$
*/
-@Test(groups = {"functional"}, enabled = false)
+@Test(groups = {"functional"}, enabled = false, description = "Disabled because this test depends on JBCACHE-315 being resolved.")
public class StateTransferUnderLoadTest
{
Cache<Object, Object> cache1, cache2;
17 years, 1 month
JBoss Cache SVN: r4721 - core/branches/1.4.X/src/org/jboss/cache.
by jbosscache-commits@lists.jboss.org
Author: manik.surtani(a)jboss.com
Date: 2007-11-01 08:15:10 -0400 (Thu, 01 Nov 2007)
New Revision: 4721
Modified:
core/branches/1.4.X/src/org/jboss/cache/TreeCache.java
Log:
JBCACHE-1209 - options not reset properly in gravitatedata
Modified: core/branches/1.4.X/src/org/jboss/cache/TreeCache.java
===================================================================
--- core/branches/1.4.X/src/org/jboss/cache/TreeCache.java 2007-11-01 12:14:07 UTC (rev 4720)
+++ core/branches/1.4.X/src/org/jboss/cache/TreeCache.java 2007-11-01 12:15:10 UTC (rev 4721)
@@ -5285,18 +5285,25 @@
throws CacheException
{
InvocationContext ctx = getInvocationContext();
+ Option opt = ctx.getOptionOverrides();
try
- {
+ {
ctx.setOriginLocal(false);
// we need to get the state for this Fqn and it's sub-nodes.
// for now, perform a very simple series of getData calls.
// use a get() call into the cache to make sure cache loading takes place.
- // no need to cache the original skipDataGravitation setting here - it will always be false of we got here!!
- ctx.getOptionOverrides().setSkipDataGravitation(true);
+ opt.setSkipDataGravitation(true);
DataNode actualNode = get(fqn);
- ctx.getOptionOverrides().setSkipDataGravitation(false);
+ // The call to get(Fqn) has changed the Option instance associated
+ // with the thread. We need to restore the state of the original option,
+ // and then restore it to the thread. This is necessary because this
+ // method can be invoked *directly* by DataGravitatorInterceptor; we need
+ // to restore whatever options were in effect when DGI was called.
+ opt.setSkipDataGravitation(false);
+ ctx.setOptionOverrides(opt);
+
Fqn backupNodeFqn = null;
if (actualNode == null && searchSubtrees)
{
@@ -5310,9 +5317,11 @@
while (childNames.hasNext() && actualNode == null)
{
backupNodeFqn = BuddyManager.getBackupFqn(childNames.next().toString(), fqn);
- ctx.getOptionOverrides().setSkipDataGravitation(true);
+ Option curOpt = ctx.getOptionOverrides();
+ curOpt.setSkipDataGravitation(true);
actualNode = get(backupNodeFqn);
- ctx.getOptionOverrides().setSkipDataGravitation(false);
+ curOpt.setSkipDataGravitation(false);
+ ctx.setOptionOverrides(opt);
}
}
}
@@ -5364,6 +5373,10 @@
finally
{
ctx.setOriginLocal(true);
+ // Make sure we restore the state of the original option,
+ // and then restore it to the thread.
+ opt.setSkipDataGravitation(false);
+ ctx.setOptionOverrides(opt);
}
}
17 years, 1 month
JBoss Cache SVN: r4720 - core/tags/1.4.1.SP6/src/org/jboss/cache.
by jbosscache-commits@lists.jboss.org
Author: manik.surtani(a)jboss.com
Date: 2007-11-01 08:14:07 -0400 (Thu, 01 Nov 2007)
New Revision: 4720
Modified:
core/tags/1.4.1.SP6/src/org/jboss/cache/TreeCache.java
Log:
JBCACHE-1209 - options in gravitatedata call not properly reset
Modified: core/tags/1.4.1.SP6/src/org/jboss/cache/TreeCache.java
===================================================================
--- core/tags/1.4.1.SP6/src/org/jboss/cache/TreeCache.java 2007-11-01 11:26:37 UTC (rev 4719)
+++ core/tags/1.4.1.SP6/src/org/jboss/cache/TreeCache.java 2007-11-01 12:14:07 UTC (rev 4720)
@@ -5285,18 +5285,25 @@
throws CacheException
{
InvocationContext ctx = getInvocationContext();
+ Option opt = ctx.getOptionOverrides();
try
{
ctx.setOriginLocal(false);
// we need to get the state for this Fqn and it's sub-nodes.
// for now, perform a very simple series of getData calls.
- // use a get() call into the cache to make sure cache loading takes place.
- // no need to cache the original skipDataGravitation setting here - it will always be false of we got here!!
- ctx.getOptionOverrides().setSkipDataGravitation(true);
+ // use a get() call into the cache to make sure cache loading takes place.
+ opt.setSkipDataGravitation(true);
DataNode actualNode = get(fqn);
- ctx.getOptionOverrides().setSkipDataGravitation(false);
+ // The call to get(Fqn) has changed the Option instance associated
+ // with the thread. We need to restore the state of the original option,
+ // and then restore it to the thread. This is necessary because this
+ // method can be invoked *directly* by DataGravitatorInterceptor; we need
+ // to restore whatever options were in effect when DGI was called.
+ opt.setSkipDataGravitation(false);
+ ctx.setOptionOverrides(opt);
+
Fqn backupNodeFqn = null;
if (actualNode == null && searchSubtrees)
{
@@ -5310,9 +5317,11 @@
while (childNames.hasNext() && actualNode == null)
{
backupNodeFqn = BuddyManager.getBackupFqn(childNames.next().toString(), fqn);
- ctx.getOptionOverrides().setSkipDataGravitation(true);
+ Option curOpt = ctx.getOptionOverrides();
+ curOpt.setSkipDataGravitation(true);
actualNode = get(backupNodeFqn);
- ctx.getOptionOverrides().setSkipDataGravitation(false);
+ curOpt.setSkipDataGravitation(false);
+ ctx.setOptionOverrides(opt);
}
}
}
@@ -5364,6 +5373,10 @@
finally
{
ctx.setOriginLocal(true);
+ // Make sure we restore the state of the original option,
+ // and then restore it to the thread.
+ opt.setSkipDataGravitation(false);
+ ctx.setOptionOverrides(opt);
}
}
17 years, 1 month
JBoss Cache SVN: r4719 - core/trunk/src/test/java/org/jboss/cache/jmx.
by jbosscache-commits@lists.jboss.org
Author: mircea.markus
Date: 2007-11-01 07:26:37 -0400 (Thu, 01 Nov 2007)
New Revision: 4719
Modified:
core/trunk/src/test/java/org/jboss/cache/jmx/LegacyConfigurationTest.java
Log:
reenable test: fails because of an issue in JGroups 2.6.CR1
Modified: core/trunk/src/test/java/org/jboss/cache/jmx/LegacyConfigurationTest.java
===================================================================
--- core/trunk/src/test/java/org/jboss/cache/jmx/LegacyConfigurationTest.java 2007-10-31 22:10:53 UTC (rev 4718)
+++ core/trunk/src/test/java/org/jboss/cache/jmx/LegacyConfigurationTest.java 2007-11-01 11:26:37 UTC (rev 4719)
@@ -228,7 +228,6 @@
}
@SuppressWarnings("unchecked")
- @Test (enabled = false) // TODO: Investigate why this fails when run outside of an IDE.
public void testLegacyMuxChannelCreation() throws Exception
{
CacheJmxWrapperMBean wrapper = new CacheJmxWrapper();
17 years, 1 month