Browse Source

Remove usage of Slf4J.

Closes #2009
pull/1956/head
Jens Schauder 9 months ago
parent
commit
ba9021243c
No known key found for this signature in database
GPG Key ID: 74F6C554AE971567
  1. 12
      spring-data-jdbc/src/main/java/org/springframework/data/jdbc/core/convert/RowDocumentResultSetExtractor.java
  2. 6
      spring-data-r2dbc/src/test/java/org/springframework/data/r2dbc/testing/ExternalDatabase.java

12
spring-data-jdbc/src/main/java/org/springframework/data/jdbc/core/convert/RowDocumentResultSetExtractor.java

@ -22,8 +22,8 @@ import java.sql.SQLException;
import java.util.Iterator; import java.util.Iterator;
import java.util.Map; import java.util.Map;
import org.slf4j.Logger; import org.apache.commons.logging.Log;
import org.slf4j.LoggerFactory; import org.apache.commons.logging.LogFactory;
import org.springframework.dao.DataRetrievalFailureException; import org.springframework.dao.DataRetrievalFailureException;
import org.springframework.data.jdbc.core.convert.RowDocumentExtractorSupport.AggregateContext; import org.springframework.data.jdbc.core.convert.RowDocumentExtractorSupport.AggregateContext;
import org.springframework.data.jdbc.core.convert.RowDocumentExtractorSupport.RowDocumentSink; import org.springframework.data.jdbc.core.convert.RowDocumentExtractorSupport.RowDocumentSink;
@ -45,8 +45,8 @@ import org.springframework.util.LinkedCaseInsensitiveMap;
*/ */
class RowDocumentResultSetExtractor { class RowDocumentResultSetExtractor {
private static final Logger log = LoggerFactory.getLogger(RowDocumentResultSetExtractor.class); private static final Log log = LogFactory.getLog(RowDocumentResultSetExtractor.class);
public static final String DUPLICATE_COLUMN_WARNING = "ResultSet contains column \"{}\" multiple times. Later column index is {}"; public static final String DUPLICATE_COLUMN_WARNING = "ResultSet contains column \"%s\" multiple times. Later column index is %s";
private final RelationalMappingContext context; private final RelationalMappingContext context;
private final PathToColumnMapping propertyToColumn; private final PathToColumnMapping propertyToColumn;
@ -76,7 +76,7 @@ class RowDocumentResultSetExtractor {
String columnName = JdbcUtils.lookupColumnName(md, i+1); String columnName = JdbcUtils.lookupColumnName(md, i+1);
Object old = document.putIfAbsent(columnName, rsv instanceof Array a ? a.getArray() : rsv); Object old = document.putIfAbsent(columnName, rsv instanceof Array a ? a.getArray() : rsv);
if (old != null) { if (old != null) {
log.warn(DUPLICATE_COLUMN_WARNING, columnName, i); log.warn(DUPLICATE_COLUMN_WARNING.formatted(columnName, i));
} }
} }
@ -120,7 +120,7 @@ class RowDocumentResultSetExtractor {
String columnLabel = metaData.getColumnLabel(i + 1); String columnLabel = metaData.getColumnLabel(i + 1);
Object old = columns.put(columnLabel, i + 1); Object old = columns.put(columnLabel, i + 1);
if (old != null) { if (old != null) {
log.warn(DUPLICATE_COLUMN_WARNING, columnLabel, i); log.warn(DUPLICATE_COLUMN_WARNING.formatted( columnLabel, i));
} }
} }
return columns; return columns;

6
spring-data-r2dbc/src/test/java/org/springframework/data/r2dbc/testing/ExternalDatabase.java

@ -23,8 +23,8 @@ import java.util.concurrent.TimeUnit;
import org.junit.jupiter.api.extension.BeforeAllCallback; import org.junit.jupiter.api.extension.BeforeAllCallback;
import org.junit.jupiter.api.extension.ExtensionContext; import org.junit.jupiter.api.extension.ExtensionContext;
import org.opentest4j.TestAbortedException; import org.opentest4j.TestAbortedException;
import org.slf4j.Logger; import org.apache.commons.logging.Log;
import org.slf4j.LoggerFactory; import org.apache.commons.logging.LogFactory;
import org.testcontainers.containers.JdbcDatabaseContainer; import org.testcontainers.containers.JdbcDatabaseContainer;
/** /**
@ -35,7 +35,7 @@ import org.testcontainers.containers.JdbcDatabaseContainer;
*/ */
public abstract class ExternalDatabase implements BeforeAllCallback { public abstract class ExternalDatabase implements BeforeAllCallback {
private static final Logger LOG = LoggerFactory.getLogger(ExternalDatabase.class); private static final Log LOG = LogFactory.getLog(ExternalDatabase.class);
/** /**
* Construct an absent database that is used as {@literal null} object if no database is available. * Construct an absent database that is used as {@literal null} object if no database is available.

Loading…
Cancel
Save