From cdff3850aecbd95608003c89d2f7b9dcffaea6d4 Mon Sep 17 00:00:00 2001 From: Jens Schauder Date: Wed, 17 Nov 2021 14:32:01 +0100 Subject: [PATCH] Migrate off Slf4J to JCL. Closes #1091 Original pull request #1092 --- .../jdbc/core/convert/BasicJdbcConverter.java | 6 +++--- .../jdbc/core/convert/ResultSetAccessor.java | 8 ++++---- .../mybatis/MyBatisDataAccessStrategy.java | 6 +++--- .../config/AbstractJdbcConfiguration.java | 6 +++--- .../jdbc/testing/DataSourceConfiguration.java | 6 +++--- .../OracleDataSourceConfiguration.java | 6 +++--- .../AbstractRelationalEventListener.java | 20 +++++++++---------- 7 files changed, 29 insertions(+), 29 deletions(-) diff --git a/spring-data-jdbc/src/main/java/org/springframework/data/jdbc/core/convert/BasicJdbcConverter.java b/spring-data-jdbc/src/main/java/org/springframework/data/jdbc/core/convert/BasicJdbcConverter.java index 0720496f..dfefccd3 100644 --- a/spring-data-jdbc/src/main/java/org/springframework/data/jdbc/core/convert/BasicJdbcConverter.java +++ b/spring-data-jdbc/src/main/java/org/springframework/data/jdbc/core/convert/BasicJdbcConverter.java @@ -22,8 +22,8 @@ import java.sql.SQLException; import java.util.Map; import java.util.Optional; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; import org.springframework.context.ApplicationContext; import org.springframework.context.ApplicationContextAware; @@ -71,7 +71,7 @@ import org.springframework.util.Assert; */ public class BasicJdbcConverter extends BasicRelationalConverter implements JdbcConverter, ApplicationContextAware { - private static final Logger LOG = LoggerFactory.getLogger(BasicJdbcConverter.class); + private static final Log LOG = LogFactory.getLog(BasicJdbcConverter.class); private static final Converter, Map> ITERABLE_OF_ENTRY_TO_MAP_CONVERTER = new IterableOfEntryToMapConverter(); private final JdbcTypeFactory typeFactory; diff --git a/spring-data-jdbc/src/main/java/org/springframework/data/jdbc/core/convert/ResultSetAccessor.java b/spring-data-jdbc/src/main/java/org/springframework/data/jdbc/core/convert/ResultSetAccessor.java index d6cc8346..26bf7d54 100644 --- a/spring-data-jdbc/src/main/java/org/springframework/data/jdbc/core/convert/ResultSetAccessor.java +++ b/spring-data-jdbc/src/main/java/org/springframework/data/jdbc/core/convert/ResultSetAccessor.java @@ -20,8 +20,8 @@ import java.sql.ResultSetMetaData; import java.sql.SQLException; import java.util.Map; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; import org.springframework.data.mapping.MappingException; import org.springframework.jdbc.support.JdbcUtils; import org.springframework.lang.Nullable; @@ -38,7 +38,7 @@ import org.springframework.util.LinkedCaseInsensitiveMap; */ class ResultSetAccessor { - private static final Logger LOG = LoggerFactory.getLogger(ResultSetAccessor.class); + private static final Log LOG = LogFactory.getLog(ResultSetAccessor.class); private final ResultSet resultSet; @@ -64,7 +64,7 @@ class ResultSetAccessor { String label = metaData.getColumnLabel(i); if (index.containsKey(label)) { - LOG.warn("ResultSet contains {} multiple times", label); + LOG.warn(String.format("ResultSet contains %s multiple times", label)); continue; } diff --git a/spring-data-jdbc/src/main/java/org/springframework/data/jdbc/mybatis/MyBatisDataAccessStrategy.java b/spring-data-jdbc/src/main/java/org/springframework/data/jdbc/mybatis/MyBatisDataAccessStrategy.java index fe375c04..0a5569d8 100644 --- a/spring-data-jdbc/src/main/java/org/springframework/data/jdbc/mybatis/MyBatisDataAccessStrategy.java +++ b/spring-data-jdbc/src/main/java/org/springframework/data/jdbc/mybatis/MyBatisDataAccessStrategy.java @@ -24,8 +24,8 @@ import java.util.stream.Collectors; import org.apache.ibatis.session.SqlSession; import org.mybatis.spring.SqlSessionTemplate; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; import org.springframework.dao.EmptyResultDataAccessException; import org.springframework.data.domain.Pageable; import org.springframework.data.domain.Sort; @@ -66,7 +66,7 @@ import org.springframework.util.Assert; */ public class MyBatisDataAccessStrategy implements DataAccessStrategy { - private static final Logger LOG = LoggerFactory.getLogger(MyBatisDataAccessStrategy.class); + private static final Log LOG = LogFactory.getLog(MyBatisDataAccessStrategy.class); private static final String VERSION_SQL_PARAMETER_NAME_OLD = "___oldOptimisticLockingVersion"; private final SqlSession sqlSession; diff --git a/spring-data-jdbc/src/main/java/org/springframework/data/jdbc/repository/config/AbstractJdbcConfiguration.java b/spring-data-jdbc/src/main/java/org/springframework/data/jdbc/repository/config/AbstractJdbcConfiguration.java index 13ca2e86..6a7702ed 100644 --- a/spring-data-jdbc/src/main/java/org/springframework/data/jdbc/repository/config/AbstractJdbcConfiguration.java +++ b/spring-data-jdbc/src/main/java/org/springframework/data/jdbc/repository/config/AbstractJdbcConfiguration.java @@ -20,8 +20,8 @@ import java.util.Collections; import java.util.List; import java.util.Optional; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; import org.springframework.beans.BeansException; import org.springframework.beans.factory.NoSuchBeanDefinitionException; @@ -66,7 +66,7 @@ import org.springframework.jdbc.core.namedparam.NamedParameterJdbcOperations; @Configuration(proxyBeanMethods = false) public class AbstractJdbcConfiguration implements ApplicationContextAware { - private static final Logger LOG = LoggerFactory.getLogger(AbstractJdbcConfiguration.class); + private static final Log LOG = LogFactory.getLog(AbstractJdbcConfiguration.class); private ApplicationContext applicationContext; diff --git a/spring-data-jdbc/src/test/java/org/springframework/data/jdbc/testing/DataSourceConfiguration.java b/spring-data-jdbc/src/test/java/org/springframework/data/jdbc/testing/DataSourceConfiguration.java index a8eebef2..aa95969d 100644 --- a/spring-data-jdbc/src/test/java/org/springframework/data/jdbc/testing/DataSourceConfiguration.java +++ b/spring-data-jdbc/src/test/java/org/springframework/data/jdbc/testing/DataSourceConfiguration.java @@ -23,8 +23,8 @@ import java.util.concurrent.TimeUnit; import javax.sql.DataSource; import org.awaitility.Awaitility; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; import org.springframework.beans.factory.annotation.Autowired; import org.springframework.context.ApplicationContext; @@ -45,7 +45,7 @@ import org.springframework.jdbc.datasource.init.ResourceDatabasePopulator; @Configuration abstract class DataSourceConfiguration { - private static final Logger LOG = LoggerFactory.getLogger(DataSourceConfiguration.class); + private static final Log LOG = LogFactory.getLog(DataSourceConfiguration.class); @Autowired Class testClass; @Autowired Environment environment; diff --git a/spring-data-jdbc/src/test/java/org/springframework/data/jdbc/testing/OracleDataSourceConfiguration.java b/spring-data-jdbc/src/test/java/org/springframework/data/jdbc/testing/OracleDataSourceConfiguration.java index b095de9a..03be1fe8 100644 --- a/spring-data-jdbc/src/test/java/org/springframework/data/jdbc/testing/OracleDataSourceConfiguration.java +++ b/spring-data-jdbc/src/test/java/org/springframework/data/jdbc/testing/OracleDataSourceConfiguration.java @@ -17,8 +17,8 @@ package org.springframework.data.jdbc.testing; import javax.sql.DataSource; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; import org.springframework.context.annotation.Configuration; import org.springframework.context.annotation.Profile; import org.springframework.jdbc.core.JdbcTemplate; @@ -40,7 +40,7 @@ import org.testcontainers.containers.OracleContainer; @Profile("oracle") public class OracleDataSourceConfiguration extends DataSourceConfiguration { - private static final Logger LOG = LoggerFactory.getLogger(OracleDataSourceConfiguration.class); + private static final Log LOG = LogFactory.getLog(OracleDataSourceConfiguration.class); private static OracleContainer ORACLE_CONTAINER; diff --git a/spring-data-relational/src/main/java/org/springframework/data/relational/core/mapping/event/AbstractRelationalEventListener.java b/spring-data-relational/src/main/java/org/springframework/data/relational/core/mapping/event/AbstractRelationalEventListener.java index a0eece89..14f046e4 100644 --- a/spring-data-relational/src/main/java/org/springframework/data/relational/core/mapping/event/AbstractRelationalEventListener.java +++ b/spring-data-relational/src/main/java/org/springframework/data/relational/core/mapping/event/AbstractRelationalEventListener.java @@ -15,8 +15,8 @@ */ package org.springframework.data.relational.core.mapping.event; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; import org.springframework.context.ApplicationListener; import org.springframework.core.GenericTypeResolver; @@ -29,7 +29,7 @@ import org.springframework.core.GenericTypeResolver; */ public class AbstractRelationalEventListener implements ApplicationListener> { - private static final Logger LOG = LoggerFactory.getLogger(AbstractRelationalEventListener.class); + private static final Log LOG = LogFactory.getLog(AbstractRelationalEventListener.class); private final Class domainClass; @@ -80,7 +80,7 @@ public class AbstractRelationalEventListener implements ApplicationListener event) { if (LOG.isDebugEnabled()) { - LOG.debug("onBeforeConvert({})", event.getEntity()); + LOG.debug(String.format("onBeforeConvert(%s)", event.getEntity())); } } @@ -92,7 +92,7 @@ public class AbstractRelationalEventListener implements ApplicationListener event) { if (LOG.isDebugEnabled()) { - LOG.debug("onBeforeSave({})", event.getAggregateChange()); + LOG.debug(String.format("onBeforeSave(%s)", event.getAggregateChange())); } } @@ -104,7 +104,7 @@ public class AbstractRelationalEventListener implements ApplicationListener event) { if (LOG.isDebugEnabled()) { - LOG.debug("onAfterSave({})", event.getAggregateChange()); + LOG.debug(String.format("onAfterSave(%s)", event.getAggregateChange())); } } @@ -118,7 +118,7 @@ public class AbstractRelationalEventListener implements ApplicationListener event) { if (LOG.isDebugEnabled()) { - LOG.debug("onAfterLoad({})", event.getEntity()); + LOG.debug(String.format("onAfterLoad(%s)", event.getEntity())); } } @@ -131,7 +131,7 @@ public class AbstractRelationalEventListener implements ApplicationListener event) { if (LOG.isDebugEnabled()) { - LOG.debug("onAfterConvert({})", event.getEntity()); + LOG.debug(String.format("onAfterConvert(%s)", event.getEntity())); } } @@ -143,7 +143,7 @@ public class AbstractRelationalEventListener implements ApplicationListener event) { if (LOG.isDebugEnabled()) { - LOG.debug("onAfterDelete({})", event.getAggregateChange()); + LOG.debug(String.format("onAfterDelete(%s)", event.getAggregateChange())); } } @@ -155,7 +155,7 @@ public class AbstractRelationalEventListener implements ApplicationListener event) { if (LOG.isDebugEnabled()) { - LOG.debug("onBeforeDelete({})", event.getAggregateChange()); + LOG.debug(String.format("onBeforeDelete(%s)", event.getAggregateChange())); } }