User: lqd Date: 02/01/11 04:31:05 Modified: src/main/org/jboss/mq Tag: Branch_2_4 GenericConnectionFactory.java Log: Backport of HEAD changes to Branch_2_4: - use org.jboss.logging.Logger instead of log4j - add debug level message when handing out a ClientIL - bump log level of "can't connect to server" from debug to error Revision Changes Path No revision No revision 1.2.2.3 +17 -10 jbossmq/src/main/org/jboss/mq/GenericConnectionFactory.java Index: GenericConnectionFactory.java =================================================================== RCS file: /cvsroot/jboss/jbossmq/src/main/org/jboss/mq/GenericConnectionFactory.java,v retrieving revision 1.2.2.2 retrieving revision 1.2.2.3 diff -u -r1.2.2.2 -r1.2.2.3 --- GenericConnectionFactory.java 2001/11/28 06:15:27 1.2.2.2 +++ GenericConnectionFactory.java 2002/01/11 12:31:05 1.2.2.3 @@ -14,12 +14,14 @@ import org.jboss.mq.il.ServerIL; import org.jboss.mq.il.ServerILFactory; +import org.jboss.logging.Logger; + /** * The RMI implementation of the DistributedConnectionFactory object * * @author Hiram Chirino ([EMAIL PROTECTED]) * @created August 16, 2001 - * @version $Revision: 1.2.2.2 $ + * @version $Revision: 1.2.2.3 $ */ public class GenericConnectionFactory implements java.io.Serializable { @@ -35,18 +37,19 @@ // These are the keys we look for in the connection properties public final static String SERVER_IL_FACTORY_KEY = "ServerILFactory"; public final static String CLIENT_IL_SERVICE_KEY = "ClientILService"; - - static org.apache.log4j.Category cat = org.apache.log4j.Category.getInstance( GenericConnectionFactory.class ); + static Logger log = Logger.getLogger( GenericConnectionFactory.class ); /** * The constructor takes a ServerIL and the Connection Properties - * parameters, The connection properties are allways required since they are - * used to setup the ClientIL, but the ServerIL can be null if the - * connection properties defines a ServerILFactory so that the SeverIL can - * be created on the client side. The ServerIL paramter is usefull for IL - * such as RMI or the JVM IL since trying to explicity create a connection - * to them is not strait forward. + * parameters, The connection properties are allways required since + * they are used to setup the ClientIL, but the ServerIL can be null + * if the connection properties defines a ServerILFactory so that + * the SeverIL can be created on the client side. + * <p> + * The ServerIL parameter is useful for ILs such as RMI or the JVM + * IL since trying to explicity create a connection to them is not + * straight forward. * * @param server Description of Parameter * @param props Description of Parameter @@ -68,6 +71,10 @@ String clientILServiceCN = connectionProperties.getProperty( CLIENT_IL_SERVICE_KEY ); ClientILService service = ( ClientILService )Class.forName( clientILServiceCN ).newInstance(); service.init( connection, connectionProperties ); + + if ( log.isDebugEnabled() ) + log.debug("Handing out ClientIL: " + clientILServiceCN); + return service; } @@ -97,7 +104,7 @@ // used to produce multiple connections. return server.cloneServerIL(); } catch ( Exception e ) { - cat.debug( "Could not connect to the server", e ); + log.error( "Could not connect to the server", e ); throw new SpyJMSException( "Could not connect to the server", e ); } }
_______________________________________________ Jboss-development mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/jboss-development