svn commit: r377486 - /tomcat/container/tc5.5.x/modules/cluster/src/share/org/apache/catalina/cluster/mcast/McastService.java

2006-02-13 Thread fhanik 
Author: fhanik
Date: Mon Feb 13 13:04:20 2006
New Revision: 377486

URL: http://svn.apache.org/viewcvs?rev=377486view=rev
Log:
Use constants instead of hard coding values

Modified:

tomcat/container/tc5.5.x/modules/cluster/src/share/org/apache/catalina/cluster/mcast/McastService.java

Modified: 
tomcat/container/tc5.5.x/modules/cluster/src/share/org/apache/catalina/cluster/mcast/McastService.java
URL: 
http://svn.apache.org/viewcvs/tomcat/container/tc5.5.x/modules/cluster/src/share/org/apache/catalina/cluster/mcast/McastService.java?rev=377486r1=377485r2=377486view=diff
==
--- 
tomcat/container/tc5.5.x/modules/cluster/src/share/org/apache/catalina/cluster/mcast/McastService.java
 (original)
+++ 
tomcat/container/tc5.5.x/modules/cluster/src/share/org/apache/catalina/cluster/mcast/McastService.java
 Mon Feb 13 13:04:20 2006
@@ -257,8 +257,8 @@
  * @throws java.lang.Exception if a IO error occurs
  */
 public void start() throws java.lang.Exception {
-start(1);
-start(2);
+start(MembershipService.MBR_RX);
+start(MembershipService.MBR_RX);
 registerMBean();
 }
 



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Re: svn commit: r377486 - /tomcat/container/tc5.5.x/modules/cluster/src/share/org/apache/catalina/cluster/mcast/McastService.java

2006-02-13 Thread Peter Rossbach 

Is that correct that you use now at McastService the same constant?

Before it was start(1) and start(2)

Peter
[EMAIL PROTECTED]



Am 13.02.2006 um 22:04 schrieb [EMAIL PROTECTED]:


Author: fhanik
Date: Mon Feb 13 13:04:20 2006
New Revision: 377486

URL: http://svn.apache.org/viewcvs?rev=377486view=rev
Log:
Use constants instead of hard coding values

Modified:
tomcat/container/tc5.5.x/modules/cluster/src/share/org/apache/ 
catalina/cluster/mcast/McastService.java


Modified: tomcat/container/tc5.5.x/modules/cluster/src/share/org/ 
apache/catalina/cluster/mcast/McastService.java
URL: http://svn.apache.org/viewcvs/tomcat/container/tc5.5.x/modules/ 
cluster/src/share/org/apache/catalina/cluster/mcast/ 
McastService.java?rev=377486r1=377485r2=377486view=diff
== 

--- tomcat/container/tc5.5.x/modules/cluster/src/share/org/apache/ 
catalina/cluster/mcast/McastService.java (original)
+++ tomcat/container/tc5.5.x/modules/cluster/src/share/org/apache/ 
catalina/cluster/mcast/McastService.java Mon Feb 13 13:04:20 2006

@@ -257,8 +257,8 @@
  * @throws java.lang.Exception if a IO error occurs
  */
 public void start() throws java.lang.Exception {
-start(1);
-start(2);
+start(MembershipService.MBR_RX);
+start(MembershipService.MBR_RX);
 registerMBean();
 }




-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]





-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Re: svn commit: r377486 - /tomcat/container/tc5.5.x/modules/cluster/src/share/org/apache/catalina/cluster/mcast/McastService.java

2006-02-13 Thread Peter Rossbach 

Ohh,

I see you have later correct this past bug. Thanks!

Are your current big changes not a very high risk to break the  
cluster functionaliy again?
Why we not going inside a branch or a cluster2 module? Currently a  
lot of user work very successfully with the tomcat 5.5.15 cluster. It  
was my fault last year to changes to quick a lot.


Regards
Peter
[EMAIL PROTECTED]



Am 14.02.2006 um 07:37 schrieb Peter Rossbach:


Is that correct that you use now at McastService the same constant?

Before it was start(1) and start(2)

Peter
[EMAIL PROTECTED]



Am 13.02.2006 um 22:04 schrieb [EMAIL PROTECTED]:


Author: fhanik
Date: Mon Feb 13 13:04:20 2006
New Revision: 377486

URL: http://svn.apache.org/viewcvs?rev=377486view=rev
Log:
Use constants instead of hard coding values

Modified:
tomcat/container/tc5.5.x/modules/cluster/src/share/org/apache/ 
catalina/cluster/mcast/McastService.java


Modified: tomcat/container/tc5.5.x/modules/cluster/src/share/org/ 
apache/catalina/cluster/mcast/McastService.java
URL: http://svn.apache.org/viewcvs/tomcat/container/tc5.5.x/ 
modules/cluster/src/share/org/apache/catalina/cluster/mcast/ 
McastService.java?rev=377486r1=377485r2=377486view=diff
= 
=
--- tomcat/container/tc5.5.x/modules/cluster/src/share/org/apache/ 
catalina/cluster/mcast/McastService.java (original)
+++ tomcat/container/tc5.5.x/modules/cluster/src/share/org/apache/ 
catalina/cluster/mcast/McastService.java Mon Feb 13 13:04:20 2006

@@ -257,8 +257,8 @@
  * @throws java.lang.Exception if a IO error occurs
  */
 public void start() throws java.lang.Exception {
-start(1);
-start(2);
+start(MembershipService.MBR_RX);
+start(MembershipService.MBR_RX);
 registerMBean();
 }




-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]





-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]





-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]