Re: [PATCH] Clustering members using UDP sockets instead of multicast ones
I agree with you. But instead of extending multicast service, I would create a completely separate membership service. I am about to change how the configuration file looks and at that point I would be more than happy to have more than one supported configuration...so give me a couple of weeks. In the meantime, why don't you just post it to the tomcat-user and who ever wants it can use it. Filip - Original Message - From: "David Cassidy" <[EMAIL PROTECTED]> To: "Tomcat Developers List" <[EMAIL PROTECTED]> Sent: Tuesday, January 27, 2004 12:22 AM Subject: RE: [PATCH] Clustering members using UDP sockets instead of multicast ones > > Filip, > > This is a very useful thing for those of us who have to contend with firewalls and multiple datacentres. > If it can't go into the main CVS area can we have a 'standard extentions' type area where things like > this can be kept with instructions on how to install ? > > Thanks > > David > > > > > > "Filip Hanik" > <[EMAIL PROTECTED]To: "Tomcat Developers List" <[EMAIL PROTECTED]> > om> cc: >Subject: RE: [PATCH] Clustering members using UDP sockets instead of multicast ones > 26/01/2004 23:30 > Please respond to > "Tomcat > Developers List" > > > > > > > thanks for the patch, I am not submitting this to CVS as I don't see that > much added value. But if others find it useful, this is configurable anyway, > so there is nothing hard coded in the current code about the membership. > hope that is ok. > Filip > > -Original Message- > From: Raul Benito [mailto:[EMAIL PROTECTED] > Sent: Monday, January 26, 2004 11:25 AM > To: [EMAIL PROTECTED]; [EMAIL PROTECTED] > Subject: [PATCH] Clustering members using UDP sockets instead of > multicast ones > > > Hi, > I'm testing the tomcat5 cluster in an environment where multicast > connections are filtered. So in order to test our implementation in a > cluster tomcat, i created this little UdpService that is heavily based > in McastService, but the cluster nodes are specified in an attribute as > comma delimited instead of been discover. It is not intended to be a > fast, it is only for testing. Perhaps somebody wants for something, and > keep doing this great work. > > Regards, > > Index: McastServiceImpl.java > === > RCS file: > /home/cvspublic/jakarta-tomcat-catalina/modules/cluster/src/share/or > g/apache/catalina/cluster/mcast/McastServiceImpl.java,v > retrieving revision 1.6 > diff -u -r1.6 McastServiceImpl.java > --- McastServiceImpl.java 13 Jan 2004 00:07:18 - 1.6 > +++ McastServiceImpl.java 26 Jan 2004 19:14:41 - > @@ -143,6 +143,11 @@ > protected long serviceStartTime = System.currentTimeMillis(); > > /** > + * void to do inheritance > + **/ > +protected McastServiceImpl() {} > + > +/** > * Create a new mcast service impl > * @param member - the local member > * @param sendFrequency - the time (ms) in between pings sent out > > > > -- > New file > jakarta-tomcat-catalina/modules/cluster/src/share/org/apache/catalin > a/cluster/mcast/UdpServiceImpl.java > > package org.apache.catalina.cluster.mcast; > > import java.net.DatagramSocket; > import java.io.IOException; > import java.net.InetAddress ; > import java.net.DatagramPacket; > import org.apache.catalina.cluster.MembershipListener; > public class UdpServiceImpl extends McastServiceImpl > { > private static org.apache.commons.logging.Log log = > org.apache.commons.logging.LogFactory.getLog( UdpService.class ); > > > > /** > * Socket that we intend to listen to > */ > protected DatagramSocket socket; > > > > /** > * The multicast address > */ > protected InetAddress[] address; > /** > * The multicast port > */ > protected int[] port; > > > > /** > * Create a new mcast service impl > * @param member - the local member > * @param sendFrequency - the time (ms) in between pings sent out > * @param expireTime - the time (ms) for a member to expire > * @param port - the mcast port > * @param bind - the bind address (not sure this is used yet) > * @param mcastAddres
RE: [PATCH] Clustering members using UDP sockets instead of multicast ones
Filip, This is a very useful thing for those of us who have to contend with firewalls and multiple datacentres. If it can't go into the main CVS area can we have a 'standard extentions' type area where things like this can be kept with instructions on how to install ? Thanks David "Filip Hanik" <[EMAIL PROTECTED]To: "Tomcat Developers List" <[EMAIL PROTECTED]> om> cc: Subject: RE: [PATCH] Clustering members using UDP sockets instead of multicast ones 26/01/2004 23:30 Please respond to "Tomcat Developers List" thanks for the patch, I am not submitting this to CVS as I don't see that much added value. But if others find it useful, this is configurable anyway, so there is nothing hard coded in the current code about the membership. hope that is ok. Filip -Original Message- From: Raul Benito [mailto:[EMAIL PROTECTED] Sent: Monday, January 26, 2004 11:25 AM To: [EMAIL PROTECTED]; [EMAIL PROTECTED] Subject: [PATCH] Clustering members using UDP sockets instead of multicast ones Hi, I'm testing the tomcat5 cluster in an environment where multicast connections are filtered. So in order to test our implementation in a cluster tomcat, i created this little UdpService that is heavily based in McastService, but the cluster nodes are specified in an attribute as comma delimited instead of been discover. It is not intended to be a fast, it is only for testing. Perhaps somebody wants for something, and keep doing this great work. Regards, Index: McastServiceImpl.java === RCS file: /home/cvspublic/jakarta-tomcat-catalina/modules/cluster/src/share/or g/apache/catalina/cluster/mcast/McastServiceImpl.java,v retrieving revision 1.6 diff -u -r1.6 McastServiceImpl.java --- McastServiceImpl.java 13 Jan 2004 00:07:18 - 1.6 +++ McastServiceImpl.java 26 Jan 2004 19:14:41 - @@ -143,6 +143,11 @@ protected long serviceStartTime = System.currentTimeMillis(); /** + * void to do inheritance + **/ +protected McastServiceImpl() {} + +/** * Create a new mcast service impl * @param member - the local member * @param sendFrequency - the time (ms) in between pings sent out -- New file jakarta-tomcat-catalina/modules/cluster/src/share/org/apache/catalin a/cluster/mcast/UdpServiceImpl.java package org.apache.catalina.cluster.mcast; import java.net.DatagramSocket; import java.io.IOException; import java.net.InetAddress ; import java.net.DatagramPacket; import org.apache.catalina.cluster.MembershipListener; public class UdpServiceImpl extends McastServiceImpl { private static org.apache.commons.logging.Log log = org.apache.commons.logging.LogFactory.getLog( UdpService.class ); /** * Socket that we intend to listen to */ protected DatagramSocket socket; /** * The multicast address */ protected InetAddress[] address; /** * The multicast port */ protected int[] port; /** * Create a new mcast service impl * @param member - the local member * @param sendFrequency - the time (ms) in between pings sent out * @param expireTime - the time (ms) for a member to expire * @param port - the mcas
RE: [PATCH] Clustering members using UDP sockets instead of multicast ones
thanks for the patch, I am not submitting this to CVS as I don't see that much added value. But if others find it useful, this is configurable anyway, so there is nothing hard coded in the current code about the membership. hope that is ok. Filip -Original Message- From: Raul Benito [mailto:[EMAIL PROTECTED] Sent: Monday, January 26, 2004 11:25 AM To: [EMAIL PROTECTED]; [EMAIL PROTECTED] Subject: [PATCH] Clustering members using UDP sockets instead of multicast ones Hi, I'm testing the tomcat5 cluster in an environment where multicast connections are filtered. So in order to test our implementation in a cluster tomcat, i created this little UdpService that is heavily based in McastService, but the cluster nodes are specified in an attribute as comma delimited instead of been discover. It is not intended to be a fast, it is only for testing. Perhaps somebody wants for something, and keep doing this great work. Regards, Index: McastServiceImpl.java === RCS file: /home/cvspublic/jakarta-tomcat-catalina/modules/cluster/src/share/or g/apache/catalina/cluster/mcast/McastServiceImpl.java,v retrieving revision 1.6 diff -u -r1.6 McastServiceImpl.java --- McastServiceImpl.java 13 Jan 2004 00:07:18 - 1.6 +++ McastServiceImpl.java 26 Jan 2004 19:14:41 - @@ -143,6 +143,11 @@ protected long serviceStartTime = System.currentTimeMillis(); /** + * void to do inheritance + **/ +protected McastServiceImpl() {} + +/** * Create a new mcast service impl * @param member - the local member * @param sendFrequency - the time (ms) in between pings sent out -- New file jakarta-tomcat-catalina/modules/cluster/src/share/org/apache/catalin a/cluster/mcast/UdpServiceImpl.java package org.apache.catalina.cluster.mcast; import java.net.DatagramSocket; import java.io.IOException; import java.net.InetAddress ; import java.net.DatagramPacket; import org.apache.catalina.cluster.MembershipListener; public class UdpServiceImpl extends McastServiceImpl { private static org.apache.commons.logging.Log log = org.apache.commons.logging.LogFactory.getLog( UdpService.class ); /** * Socket that we intend to listen to */ protected DatagramSocket socket; /** * The multicast address */ protected InetAddress[] address; /** * The multicast port */ protected int[] port; /** * Create a new mcast service impl * @param member - the local member * @param sendFrequency - the time (ms) in between pings sent out * @param expireTime - the time (ms) for a member to expire * @param port - the mcast port * @param bind - the bind address (not sure this is used yet) * @param mcastAddress - the mcast address * @param service - the callback service * @throws IOException */ public UdpServiceImpl( McastMember member, long sendFrequency, long expireTime, int port, int[] ports, InetAddress bind, InetAddress[] mcastAddress, MembershipListener service) throws IOException { System.out.println("HOLA0"); if ( bind != null) socket = new DatagramSocket(new java.net.InetSocketAddress(bind,port)); else socket = new DatagramSocket(port); this.member = member; address = mcastAddress; this.port = ports; receivePacket = new DatagramPacket(new byte[1000],1000); //receivePacket.setAddress(address); receivePacket.setPort(port); membership = new McastMembership(member.getName()); timeToExpiration = expireTime; this.service = service; this.sendFrequency = sendFrequency; } /** * Send a ping * @throws Exception */ public void send() throws Exception{ member.inc(); byte[] data = member.getData(this.serviceStartTime); for (int i=0;i