[jboss-user] [Clustering/JBoss] - Re: Cluster is loosing his master

[bernd.koecke]

I posted two Bugreports:

JBAS-4313 
JBAS-4314

But I found a minor bug in HAPartitionImpl. The debug output in method viewAccepted doesn't log the old size, because the members-field changed during the method. I changed 'this.memebers' to 'oldMembers' with some checking for oldMembers == null. Its not very elegant, but it works. Should I post a bug for this, too? I think it's not so important. 

diff -u:

--- HAPartitionImpl.java        2007-04-10 18:30:52.000000000 +0200
+++ HAPartitionImpl.java-patched        2007-04-10 18:38:51.000000000 +0200
@@ -503,7 +503,10 @@
             event.originatingGroups = mergeView.getSubgroups();
          }

-         log.debug("membership changed from " + this.members.size() + " to "
+         //log.debug("membership changed from " + this.members.size() + " to "
+         int tmpOldSize = 0;
+         if (oldMembers != null) tmpOldSize = oldMembers.size();
+         log.debug("membership changed from " + tmpOldSize + " to "
             + event.allMembers.size());
          // Put the view change to the asynch queue
          this.asynchViewChanges.put(event);


View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4036067#4036067

Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4036067