[jbosscache-commits] JBoss Cache SVN: r8247 - core/trunk/src/main/java/org/jboss/cache/buddyreplication.

[jbosscache-commits at lists.jboss.org]

Author: bstansberry at jboss.com
Date: 2009-10-12 20:23:30 -0400 (Mon, 12 Oct 2009)
New Revision: 8247

Modified:
   core/trunk/src/main/java/org/jboss/cache/buddyreplication/BuddyManager.java
Log:
[JBCACHE-1549] 

Modified: core/trunk/src/main/java/org/jboss/cache/buddyreplication/BuddyManager.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/buddyreplication/BuddyManager.java	2009-10-12 22:30:16 UTC (rev 8246)
+++ core/trunk/src/main/java/org/jboss/cache/buddyreplication/BuddyManager.java	2009-10-13 00:23:30 UTC (rev 8247)
@@ -1263,13 +1263,16 @@
             return;
          }
 
-         // there is a strange case where JGroups issues view changes and just includes self in new views, and then
-         // quickly corrects it.  Happens intermittently on some unit tests.  If this is such a case, please ignore.
-         if (members.newMembers.size() == 1 && members.newMembers.get(0).equals(cache.getLocalAddress()))
-         {
-            log.info("Ignoring membership change event since it only contains self.");
-            return;
-         }
+         // BES 2009/10/12 This next bit is highly suspect and causes
+         // JBCACHE-1549, so I've commented it out
+         
+//         // there is a strange case where JGroups issues view changes and just includes self in new views, and then
+//         // quickly corrects it.  Happens intermittently on some unit tests.  If this is such a case, please ignore.
+//         if (members.newMembers.size() == 1 && members.newMembers.get(0).equals(cache.getLocalAddress()))
+//         {
+//            log.info("Ignoring membership change event since it only contains self.");
+//            return;
+//         }
 
          broadcastPoolMembership(members);