Remove usage of Slf4J.

Closes #2009

Author: schauder

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

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

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

@@ -23,8 +23,8 @@
 import org.junit.jupiter.api.extension.BeforeAllCallback;
 import org.junit.jupiter.api.extension.ExtensionContext;
 import org.opentest4j.TestAbortedException;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
 import org.testcontainers.containers.JdbcDatabaseContainer;
 
 /**
@@ -35,7 +35,7 @@
  */
 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.