Author: steve.ebersole(a)jboss.com Date: 2007-08-03 13:42:55 -0400 (Fri, 03 Aug 2007) New Revision: 12897 Modified: core/trunk/connection-proxool/src/main/java/org/hibernate/connection/ProxoolConnectionProvider.java Log: HHH-2696 : migrate to slf4j Modified: core/trunk/connection-proxool/src/main/java/org/hibernate/connection/ProxoolConnectionProvider.java =================================================================== --- core/trunk/connection-proxool/src/main/java/org/hibernate/connection/ProxoolConnectionProvider.java 2007-08-03 17:39:19 UTC (rev 12896) +++ core/trunk/connection-proxool/src/main/java/org/hibernate/connection/ProxoolConnectionProvider.java 2007-08-03 17:42:55 UTC (rev 12897) @@ -90,10 +90,10 @@ if ( "true".equals(externalConfig) ) { // Validate that an alias name was provided to determine which pool to use - if ( !StringHelper.isNotEmpty(proxoolAlias) ) { + if ( !StringHelper.isNotEmpty( proxoolAlias ) ) { String msg = "Cannot configure Proxool Provider to use an existing in memory pool without the " + Environment.PROXOOL_POOL_ALIAS + " property set."; - log.fatal(msg); - throw new HibernateException(msg); + log.error( msg ); + throw new HibernateException( msg ); } // Append the stem to the proxool pool alias proxoolAlias = PROXOOL_JDBC_STEM + proxoolAlias; @@ -101,28 +101,28 @@ // Set the existing pool flag to true existingPool = true; - log.info("Configuring Proxool Provider using existing pool in memory: " + proxoolAlias); + log.info( "Configuring Proxool Provider using existing pool in memory: " + proxoolAlias ); // Configured using the JAXP Configurator } - else if ( StringHelper.isNotEmpty(jaxpFile) ) { + else if ( StringHelper.isNotEmpty( jaxpFile ) ) { - log.info("Configuring Proxool Provider using JAXPConfigurator: " + jaxpFile); + log.info( "Configuring Proxool Provider using JAXPConfigurator: " + jaxpFile ); // Validate that an alias name was provided to determine which pool to use - if ( !StringHelper.isNotEmpty(proxoolAlias) ) { + if ( !StringHelper.isNotEmpty( proxoolAlias ) ) { String msg = "Cannot configure Proxool Provider to use JAXP without the " + Environment.PROXOOL_POOL_ALIAS + " property set."; - log.fatal(msg); - throw new HibernateException(msg); + log.error( msg ); + throw new HibernateException( msg ); } try { - JAXPConfigurator.configure( ConfigHelper.getConfigStreamReader(jaxpFile), false ); + JAXPConfigurator.configure( ConfigHelper.getConfigStreamReader( jaxpFile ), false ); } - catch (ProxoolException e) { + catch ( ProxoolException e ) { String msg = "Proxool Provider unable to load JAXP configurator file: " + jaxpFile; - log.fatal(msg, e); - throw new HibernateException(msg, e); + log.error( msg, e ); + throw new HibernateException( msg, e ); } // Append the stem to the proxool pool alias @@ -131,24 +131,24 @@ // Configured using the Properties File Configurator } - else if ( StringHelper.isNotEmpty(propFile) ) { + else if ( StringHelper.isNotEmpty( propFile ) ) { - log.info("Configuring Proxool Provider using Properties File: " + propFile); + log.info( "Configuring Proxool Provider using Properties File: " + propFile ); // Validate that an alias name was provided to determine which pool to use - if ( !StringHelper.isNotEmpty(proxoolAlias) ) { + if ( !StringHelper.isNotEmpty( proxoolAlias ) ) { String msg = "Cannot configure Proxool Provider to use Properties File without the " + Environment.PROXOOL_POOL_ALIAS + " property set."; - log.fatal(msg); - throw new HibernateException(msg); + log.error( msg ); + throw new HibernateException( msg ); } try { - PropertyConfigurator.configure( ConfigHelper.getConfigProperties(propFile) ); + PropertyConfigurator.configure( ConfigHelper.getConfigProperties( propFile ) ); } - catch (ProxoolException e) { + catch ( ProxoolException e ) { String msg = "Proxool Provider unable to load load Property configurator file: " + propFile; - log.fatal(msg, e); - throw new HibernateException(msg, e); + log.error( msg, e ); + throw new HibernateException( msg, e ); } // Append the stem to the proxool pool alias