diff --git a/pom.xml b/pom.xml index b6410e6887..5ce0edb136 100644 --- a/pom.xml +++ b/pom.xml @@ -5,7 +5,7 @@ org.springframework.data spring-data-mongodb-parent - 3.4.0-SNAPSHOT + 3.4.0-3881-SNAPSHOT pom Spring Data MongoDB diff --git a/spring-data-mongodb-benchmarks/pom.xml b/spring-data-mongodb-benchmarks/pom.xml index e2704a6753..8d0339fc94 100644 --- a/spring-data-mongodb-benchmarks/pom.xml +++ b/spring-data-mongodb-benchmarks/pom.xml @@ -7,7 +7,7 @@ org.springframework.data spring-data-mongodb-parent - 3.4.0-SNAPSHOT + 3.4.0-3881-SNAPSHOT ../pom.xml diff --git a/spring-data-mongodb-distribution/pom.xml b/spring-data-mongodb-distribution/pom.xml index b75f8bf624..beb11e8e85 100644 --- a/spring-data-mongodb-distribution/pom.xml +++ b/spring-data-mongodb-distribution/pom.xml @@ -14,7 +14,7 @@ org.springframework.data spring-data-mongodb-parent - 3.4.0-SNAPSHOT + 3.4.0-3881-SNAPSHOT ../pom.xml diff --git a/spring-data-mongodb/pom.xml b/spring-data-mongodb/pom.xml index e3d753e766..4bad32590c 100644 --- a/spring-data-mongodb/pom.xml +++ b/spring-data-mongodb/pom.xml @@ -11,7 +11,7 @@ org.springframework.data spring-data-mongodb-parent - 3.4.0-SNAPSHOT + 3.4.0-3881-SNAPSHOT ../pom.xml @@ -237,13 +237,6 @@ true - - org.slf4j - jul-to-slf4j - ${slf4j} - test - - nl.jqno.equalsverifier equalsverifier diff --git a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/SpringDataMongoDB.java b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/SpringDataMongoDB.java index 808b576bcb..c136ade3f5 100644 --- a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/SpringDataMongoDB.java +++ b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/SpringDataMongoDB.java @@ -15,8 +15,8 @@ */ package org.springframework.data.mongodb; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; import org.springframework.data.util.Version; import org.springframework.util.StringUtils; @@ -31,7 +31,7 @@ */ public class SpringDataMongoDB { - private static final Logger LOGGER = LoggerFactory.getLogger(SpringDataMongoDB.class); + private static final Log LOGGER = LogFactory.getLog(SpringDataMongoDB.class); private static final Version FALLBACK_VERSION = new Version(3); private static final MongoDriverInformation DRIVER_INFORMATION = MongoDriverInformation @@ -68,7 +68,7 @@ public static Version version() { try { return Version.parse(versionString); } catch (Exception e) { - LOGGER.debug("Cannot read Spring Data MongoDB version '{}'.", versionString); + LOGGER.debug(String.format("Cannot read Spring Data MongoDB version '%s'.", versionString)); } return FALLBACK_VERSION; diff --git a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/config/ServerAddressPropertyEditor.java b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/config/ServerAddressPropertyEditor.java index 5e5d82f6be..34b59e7809 100644 --- a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/config/ServerAddressPropertyEditor.java +++ b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/config/ServerAddressPropertyEditor.java @@ -21,8 +21,8 @@ import java.util.HashSet; import java.util.Set; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; import org.springframework.lang.Nullable; import org.springframework.util.Assert; import org.springframework.util.StringUtils; @@ -43,8 +43,8 @@ public class ServerAddressPropertyEditor extends PropertyEditorSupport { * A port is a number without a leading 0 at the end of the address that is proceeded by just a single :. */ private static final String HOST_PORT_SPLIT_PATTERN = "(? 2) { - LOG.warn(COULD_NOT_PARSE_ADDRESS_MESSAGE, "source", source); + LOG.warn(String.format(COULD_NOT_PARSE_ADDRESS_MESSAGE, "source", source)); return null; } @@ -105,9 +105,9 @@ private ServerAddress parseServerAddress(String source) { return port == null ? new ServerAddress(hostAddress) : new ServerAddress(hostAddress, port); } catch (UnknownHostException e) { - LOG.warn(COULD_NOT_PARSE_ADDRESS_MESSAGE, "host", hostAndPort[0]); + LOG.warn(String.format(COULD_NOT_PARSE_ADDRESS_MESSAGE, "host", hostAndPort[0])); } catch (NumberFormatException e) { - LOG.warn(COULD_NOT_PARSE_ADDRESS_MESSAGE, "port", hostAndPort[1]); + LOG.warn(String.format(COULD_NOT_PARSE_ADDRESS_MESSAGE, "port", hostAndPort[1])); } return null; diff --git a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/MongoTemplate.java b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/MongoTemplate.java index 93477b738e..895097dd47 100644 --- a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/MongoTemplate.java +++ b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/MongoTemplate.java @@ -24,10 +24,10 @@ import java.util.concurrent.TimeUnit; import java.util.stream.Collectors; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; import org.bson.Document; import org.bson.conversions.Bson; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; import org.springframework.beans.BeansException; import org.springframework.context.ApplicationContext; @@ -163,7 +163,7 @@ */ public class MongoTemplate implements MongoOperations, ApplicationContextAware, IndexOperationsProvider { - private static final Logger LOGGER = LoggerFactory.getLogger(MongoTemplate.class); + private static final Log LOGGER = LogFactory.getLog(MongoTemplate.class); private static final WriteResultChecking DEFAULT_WRITE_RESULT_CHECKING = WriteResultChecking.NONE; private final MongoConverter mongoConverter; @@ -507,8 +507,8 @@ protected void executeQuery(Query query, String collectionName, DocumentCallback Document fieldsObject = query.getFieldsObject(); if (LOGGER.isDebugEnabled()) { - LOGGER.debug("Executing query: {} sort: {} fields: {} in collection: {}", serializeToJsonSafely(queryObject), - sortObject, fieldsObject, collectionName); + LOGGER.debug(String.format("Executing query: %s sort: %s fields: %s in collection: %s", + serializeToJsonSafely(queryObject), sortObject, fieldsObject, collectionName)); } this.executeQueryInternal(new FindCallback(queryObject, fieldsObject, null), @@ -700,8 +700,8 @@ public void dropCollection(String collectionName) { execute(collectionName, (CollectionCallback) collection -> { collection.drop(); if (LOGGER.isDebugEnabled()) { - LOGGER.debug("Dropped collection [{}]", - collection.getNamespace() != null ? collection.getNamespace().getCollectionName() : collectionName); + LOGGER.debug(String.format("Dropped collection [%s]", + collection.getNamespace() != null ? collection.getNamespace().getCollectionName() : collectionName)); } return null; }); @@ -903,8 +903,8 @@ public List findDistinct(Query query, String field, String collectionName MongoIterable result = execute(collectionName, (collection) -> { if (LOGGER.isDebugEnabled()) { - LOGGER.debug("Executing findDistinct using query {} for field: {} in collection: {}", - serializeToJsonSafely(mappedQuery), field, collectionName); + LOGGER.debug(String.format("Executing findDistinct using query %s for field: %s in collection: %s", + serializeToJsonSafely(mappedQuery), field, collectionName)); } QueryCursorPreparer preparer = new QueryCursorPreparer(query, entityClass); @@ -1126,7 +1126,8 @@ public long count(Query query, @Nullable Class entityClass, String collection protected long doCount(String collectionName, Document filter, CountOptions options) { if (LOGGER.isDebugEnabled()) { - LOGGER.debug("Executing count: {} in collection: {}", serializeToJsonSafely(filter), collectionName); + LOGGER + .debug(String.format("Executing count: %s in collection: %s", serializeToJsonSafely(filter), collectionName)); } return execute(collectionName, @@ -1453,7 +1454,8 @@ protected T doSave(String collectionName, T objectToSave, MongoWriter wri protected Object insertDocument(String collectionName, Document document, Class entityClass) { if (LOGGER.isDebugEnabled()) { - LOGGER.debug("Inserting Document containing fields: {} in collection: {}", document.keySet(), collectionName); + LOGGER.debug(String.format("Inserting Document containing fields: %s in collection: %s", document.keySet(), + collectionName)); } return execute(collectionName, collection -> { @@ -1478,7 +1480,7 @@ protected List insertDocumentList(String collectionName, List } if (LOGGER.isDebugEnabled()) { - LOGGER.debug("Inserting list of Documents containing {} items", documents.size()); + LOGGER.debug(String.format("Inserting list of Documents containing %s items", documents.size())); } execute(collectionName, collection -> { @@ -1502,7 +1504,7 @@ protected List insertDocumentList(String collectionName, List protected Object saveDocument(String collectionName, Document dbDoc, Class entityClass) { if (LOGGER.isDebugEnabled()) { - LOGGER.debug("Saving Document containing fields: {}", dbDoc.keySet()); + LOGGER.debug(String.format("Saving Document containing fields: %s", dbDoc.keySet())); } return execute(collectionName, collection -> { @@ -1607,8 +1609,8 @@ protected UpdateResult doUpdate(String collectionName, Query query, UpdateDefini if (query.isSorted() && LOGGER.isWarnEnabled()) { - LOGGER.warn("{} does not support sort ('{}'). Please use findAndModify() instead.", - upsert ? "Upsert" : "UpdateFirst", serializeToJsonSafely(query.getSortObject())); + LOGGER.warn(String.format("%s does not support sort ('%s'). Please use findAndModify() instead.", + upsert ? "Upsert" : "UpdateFirst", serializeToJsonSafely(query.getSortObject()))); } MongoPersistentEntity entity = entityClass == null ? null : getPersistentEntity(entityClass); @@ -1630,8 +1632,8 @@ protected UpdateResult doUpdate(String collectionName, Query query, UpdateDefini return execute(collectionName, collection -> { if (LOGGER.isDebugEnabled()) { - LOGGER.debug("Calling update using query: {} and update: {} in collection: {}", - serializeToJsonSafely(queryObj), serializeToJsonSafely(pipeline), collectionName); + LOGGER.debug(String.format("Calling update using query: %s and update: %s in collection: %s", + serializeToJsonSafely(queryObj), serializeToJsonSafely(pipeline), collectionName)); } collection = writeConcernToUse != null ? collection.withWriteConcern(writeConcernToUse) : collection; @@ -1648,8 +1650,8 @@ protected UpdateResult doUpdate(String collectionName, Query query, UpdateDefini return execute(collectionName, collection -> { if (LOGGER.isDebugEnabled()) { - LOGGER.debug("Calling update using query: {} and update: {} in collection: {}", serializeToJsonSafely(queryObj), - serializeToJsonSafely(updateObj), collectionName); + LOGGER.debug(String.format("Calling update using query: %s and update: %s in collection: %s", + serializeToJsonSafely(queryObj), serializeToJsonSafely(updateObj), collectionName)); } collection = writeConcernToUse != null ? collection.withWriteConcern(writeConcernToUse) : collection; @@ -1739,8 +1741,8 @@ protected DeleteResult doRemove(String collectionName, Query query, @Nullabl Document removeQuery = queryObject; if (LOGGER.isDebugEnabled()) { - LOGGER.debug("Remove using query: {} in collection: {}.", - new Object[] { serializeToJsonSafely(removeQuery), collectionName }); + LOGGER.debug(String.format("Remove using query: %s in collection: %s.", serializeToJsonSafely(removeQuery), + collectionName)); } if (query.getLimit() > 0 || query.getSkip() > 0) { @@ -1954,13 +1956,13 @@ public GroupByResults group(@Nullable Criteria criteria, String inputColl Document commandObject = new Document("group", document); if (LOGGER.isDebugEnabled()) { - LOGGER.debug("Executing Group with Document [{}]", serializeToJsonSafely(commandObject)); + LOGGER.debug(String.format("Executing Group with Document [%s]", serializeToJsonSafely(commandObject))); } Document commandResult = executeCommand(commandObject, this.readPreference); if (LOGGER.isDebugEnabled()) { - LOGGER.debug("Group command result = [{}]", commandResult); + LOGGER.debug(String.format("Group command result = [%s]", commandResult)); } @SuppressWarnings("unchecked") @@ -2132,7 +2134,7 @@ protected AggregationResults doAggregate(Aggregation aggregation, String Document command = aggregationUtil.createCommand(collectionName, aggregation, context); if (LOGGER.isDebugEnabled()) { - LOGGER.debug("Executing aggregation: {}", serializeToJsonSafely(command)); + LOGGER.debug(String.format("Executing aggregation: %s", serializeToJsonSafely(command))); } Document commandResult = executeCommand(command); @@ -2143,7 +2145,8 @@ protected AggregationResults doAggregate(Aggregation aggregation, String List pipeline = aggregationUtil.createPipeline(aggregation, context); if (LOGGER.isDebugEnabled()) { - LOGGER.debug("Executing aggregation: {} in collection {}", serializeToJsonSafely(pipeline), collectionName); + LOGGER.debug( + String.format("Executing aggregation: %s in collection %s", serializeToJsonSafely(pipeline), collectionName)); } return execute(collectionName, collection -> { @@ -2209,7 +2212,8 @@ protected CloseableIterator aggregateStream(Aggregation aggregation, Stri List pipeline = aggregationDefinition.getAggregationPipeline(); if (LOGGER.isDebugEnabled()) { - LOGGER.debug("Streaming aggregation: {} in collection {}", serializeToJsonSafely(pipeline), collectionName); + LOGGER.debug( + String.format("Streaming aggregation: %s in collection %s", serializeToJsonSafely(pipeline), collectionName)); } ReadDocumentCallback readCallback = new ReadDocumentCallback<>(mongoConverter, outputType, collectionName); @@ -2455,8 +2459,8 @@ protected MongoCollection doCreateCollection(String collectionName, Do // TODO: Emit a collection created event if (LOGGER.isDebugEnabled()) { - LOGGER.debug("Created collection [{}]", - coll.getNamespace() != null ? coll.getNamespace().getCollectionName() : collectionName); + LOGGER.debug(String.format("Created collection [%s]", + coll.getNamespace() != null ? coll.getNamespace().getCollectionName() : collectionName)); } return coll; }); @@ -2499,8 +2503,8 @@ protected T doFindOne(String collectionName, Document query, Document fields Document mappedQuery = queryContext.getMappedQuery(entity); if (LOGGER.isDebugEnabled()) { - LOGGER.debug("findOne using query: {} fields: {} for class: {} in collection: {}", serializeToJsonSafely(query), - mappedFields, entityClass, collectionName); + LOGGER.debug(String.format("findOne using query: %s fields: %s for class: %s in collection: %s", + serializeToJsonSafely(query), mappedFields, entityClass, collectionName)); } return executeFindOneInternal(new FindOneCallback(mappedQuery, mappedFields, preparer), @@ -2551,8 +2555,8 @@ protected List doFind(String collectionName, Document query, Document Document mappedQuery = queryContext.getMappedQuery(entity); if (LOGGER.isDebugEnabled()) { - LOGGER.debug("find using query: {} fields: {} for class: {} in collection: {}", - serializeToJsonSafely(mappedQuery), mappedFields, entityClass, collectionName); + LOGGER.debug(String.format("find using query: %s fields: %s for class: %s in collection: %s", + serializeToJsonSafely(mappedQuery), mappedFields, entityClass, collectionName)); } return executeFindMultiInternal(new FindCallback(mappedQuery, mappedFields, null), @@ -2575,8 +2579,8 @@ List doFind(String collectionName, Document query, Document fields, Cl Document mappedQuery = queryContext.getMappedQuery(entity); if (LOGGER.isDebugEnabled()) { - LOGGER.debug("find using query: {} fields: {} for class: {} in collection: {}", - serializeToJsonSafely(mappedQuery), mappedFields, sourceClass, collectionName); + LOGGER.debug(String.format("find using query: %s fields: %s for class: %s in collection: %s", + serializeToJsonSafely(mappedQuery), mappedFields, sourceClass, collectionName)); } return executeFindMultiInternal(new FindCallback(mappedQuery, mappedFields, null), preparer, @@ -2678,8 +2682,8 @@ protected T doFindAndRemove(String collectionName, Document query, Document EntityReader readerToUse = this.mongoConverter; if (LOGGER.isDebugEnabled()) { - LOGGER.debug("findAndRemove using query: {} fields: {} sort: {} for class: {} in collection: {}", - serializeToJsonSafely(query), fields, sort, entityClass, collectionName); + LOGGER.debug(String.format("findAndRemove using query: %s fields: %s sort: %s for class: %s in collection: %s", + serializeToJsonSafely(query), fields, sort, entityClass, collectionName)); } MongoPersistentEntity entity = mappingContext.getPersistentEntity(entityClass); @@ -2709,10 +2713,10 @@ protected T doFindAndModify(String collectionName, Document query, Document : updateContext.getMappedUpdate(entity); if (LOGGER.isDebugEnabled()) { - LOGGER.debug( - "findAndModify using query: {} fields: {} sort: {} for class: {} and update: {} " + "in collection: {}", + LOGGER.debug(String.format( + "findAndModify using query: %s fields: %s sort: %s for class: %s and update: %s in collection: %s", serializeToJsonSafely(mappedQuery), fields, sort, entityClass, serializeToJsonSafely(mappedUpdate), - collectionName); + collectionName)); } return executeFindOneInternal( @@ -2742,10 +2746,10 @@ protected T doFindAndReplace(String collectionName, Document mappedQuery, Do Document replacement, FindAndReplaceOptions options, Class resultType) { if (LOGGER.isDebugEnabled()) { - LOGGER.debug( - "findAndReplace using query: {} fields: {} sort: {} for class: {} and replacement: {} " + "in collection: {}", + LOGGER.debug(String.format( + "findAndReplace using query: %s fields: %s sort: %s for class: %s and replacement: %s " + "in collection: %s", serializeToJsonSafely(mappedQuery), serializeToJsonSafely(mappedFields), serializeToJsonSafely(mappedSort), - entityType, serializeToJsonSafely(replacement), collectionName); + entityType, serializeToJsonSafely(replacement), collectionName)); } return executeFindOneInternal( @@ -2937,9 +2941,10 @@ public Document doInCollection(MongoCollection collection) throws Mong if (LOGGER.isDebugEnabled()) { - LOGGER.debug("findOne using query: {} fields: {} in db.collection: {}", serializeToJsonSafely(query), + LOGGER.debug(String.format("findOne using query: %s fields: %s in db.collection: %s", + serializeToJsonSafely(query), serializeToJsonSafely(fields.orElseGet(Document::new)), - collection.getNamespace() != null ? collection.getNamespace().getFullName() : "n/a"); + collection.getNamespace() != null ? collection.getNamespace().getFullName() : "n/a")); } if (fields.isPresent()) { diff --git a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/ReactiveMongoTemplate.java b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/ReactiveMongoTemplate.java index a7d0113e8f..e408e62cdc 100644 --- a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/ReactiveMongoTemplate.java +++ b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/ReactiveMongoTemplate.java @@ -36,14 +36,14 @@ import java.util.function.Function; import java.util.stream.Collectors; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; import org.bson.BsonValue; import org.bson.Document; import org.bson.conversions.Bson; import org.bson.types.ObjectId; import org.reactivestreams.Publisher; import org.reactivestreams.Subscriber; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; import org.springframework.beans.BeansException; import org.springframework.context.ApplicationContext; @@ -165,7 +165,7 @@ public class ReactiveMongoTemplate implements ReactiveMongoOperations, Applicati public static final DbRefResolver NO_OP_REF_RESOLVER = NoOpDbRefResolver.INSTANCE; - private static final Logger LOGGER = LoggerFactory.getLogger(ReactiveMongoTemplate.class); + private static final Log LOGGER = LogFactory.getLog(ReactiveMongoTemplate.class); private static final WriteResultChecking DEFAULT_WRITE_RESULT_CHECKING = WriteResultChecking.NONE; private final MongoConverter mongoConverter; @@ -829,7 +829,7 @@ public Mono exists(Query query, @Nullable Class entityClass, String .projection(new Document("_id", 1)); if (LOGGER.isDebugEnabled()) { - LOGGER.debug("exists: {} in collection: {}", serializeToJsonSafely(filter), collectionName); + LOGGER.debug(String.format("exists: %s in collection: %s", serializeToJsonSafely(filter), collectionName)); } queryContext.applyCollation(entityClass, findPublisher::collation); @@ -911,8 +911,8 @@ public Flux findDistinct(Query query, String field, String collectionName Flux result = execute(collectionName, collection -> { if (LOGGER.isDebugEnabled()) { - LOGGER.debug("Executing findDistinct using query {} for field: {} in collection: {}", - serializeToJsonSafely(mappedQuery), field, collectionName); + LOGGER.debug(String.format("Executing findDistinct using query %s for field: %s in collection: %s", + serializeToJsonSafely(mappedQuery), field, collectionName)); } FindPublisherPreparer preparer = new QueryFindPublisherPreparer(query, entityClass); @@ -988,8 +988,8 @@ protected Flux doAggregate(Aggregation aggregation, String collectionName AggregationDefinition ctx = queryOperations.createAggregation(aggregation, inputType); if (LOGGER.isDebugEnabled()) { - LOGGER.debug("Streaming aggregation: {} in collection {}", serializeToJsonSafely(ctx.getAggregationPipeline()), - collectionName); + LOGGER.debug(String.format("Streaming aggregation: %s in collection %s", + serializeToJsonSafely(ctx.getAggregationPipeline()), collectionName)); } ReadDocumentCallback readCallback = new ReadDocumentCallback<>(mongoConverter, outputType, collectionName); @@ -1226,7 +1226,8 @@ public Mono count(Query query, @Nullable Class entityClass, String coll Document filter = countContext.getMappedQuery(entityClass, mappingContext::getPersistentEntity); if (LOGGER.isDebugEnabled()) { - LOGGER.debug("Executing count: {} in collection: {}", serializeToJsonSafely(filter), collectionName); + LOGGER.debug( + String.format("Executing count: %s in collection: %s", serializeToJsonSafely(filter), collectionName)); } return doCount(collectionName, filter, options); @@ -1556,7 +1557,8 @@ protected Mono doSave(String collectionName, T objectToSave, MongoWriter< protected Mono insertDocument(String collectionName, Document dbDoc, Class entityClass) { if (LOGGER.isDebugEnabled()) { - LOGGER.debug("Inserting Document containing fields: " + dbDoc.keySet() + " in collection: " + collectionName); + LOGGER.debug(String + .format("Inserting Document containing fields: " + dbDoc.keySet() + " in collection: " + collectionName)); } Document document = new Document(dbDoc); @@ -1582,7 +1584,7 @@ protected Flux insertDocumentList(String collectionName, List documents = new ArrayList<>(); @@ -1620,7 +1622,7 @@ private MongoCollection prepareCollection(MongoCollection co protected Mono saveDocument(String collectionName, Document document, Class entityClass) { if (LOGGER.isDebugEnabled()) { - LOGGER.debug("Saving Document containing fields: " + document.keySet()); + LOGGER.debug(String.format("Saving Document containing fields: %s", document.keySet())); } return createMono(collectionName, collection -> { @@ -1747,8 +1749,8 @@ protected Mono doUpdate(String collectionName, Query query, @Nulla if (query.isSorted() && LOGGER.isWarnEnabled()) { - LOGGER.warn("{} does not support sort ('{}'). Please use findAndModify() instead.", - upsert ? "Upsert" : "UpdateFirst", serializeToJsonSafely(query.getSortObject())); + LOGGER.warn(String.format("%s does not support sort ('%s'). Please use findAndModify() instead.", + upsert ? "Upsert" : "UpdateFirst", serializeToJsonSafely(query.getSortObject()))); } MongoPersistentEntity entity = entityClass == null ? null : getPersistentEntity(entityClass); @@ -1959,8 +1961,8 @@ protected Mono doRemove(String collectionName, Query query, @N MongoCollection collectionToUse = prepareCollection(collection, writeConcernToUse); if (LOGGER.isDebugEnabled()) { - LOGGER.debug("Remove using query: {} in collection: {}.", - new Object[] { serializeToJsonSafely(removeQuery), collectionName }); + LOGGER.debug(String.format("Remove using query: %s in collection: %s.", serializeToJsonSafely(removeQuery), + collectionName)); } if (query.getLimit() > 0 || query.getSkip() > 0) { @@ -2328,7 +2330,7 @@ protected Mono> doCreateCollection(String collectionNa // TODO: Emit a collection created event if (LOGGER.isDebugEnabled()) { - LOGGER.debug("Created collection [{}]", collectionName); + LOGGER.debug(String.format("Created collection [%s]", collectionName)); } }).then(getCollection(collectionName)); @@ -2451,8 +2453,8 @@ Flux doFind(String collectionName, Document query, Document fields, Cl Document mappedQuery = queryContext.getMappedQuery(entity); if (LOGGER.isDebugEnabled()) { - LOGGER.debug("find using query: {} fields: {} for class: {} in collection: {}", - serializeToJsonSafely(mappedQuery), mappedFields, sourceClass, collectionName); + LOGGER.debug(String.format("find using query: %s fields: %s for class: %s in collection: %s", + serializeToJsonSafely(mappedQuery), mappedFields, sourceClass, collectionName)); } return executeFindMultiInternal(new FindCallback(mappedQuery, mappedFields), preparer, @@ -2611,11 +2613,11 @@ protected Mono doFindAndReplace(String collectionName, Document mappedQue return Mono.defer(() -> { if (LOGGER.isDebugEnabled()) { - LOGGER.debug( - "findAndReplace using query: {} fields: {} sort: {} for class: {} and replacement: {} " - + "in collection: {}", + LOGGER.debug(String.format( + "findAndReplace using query: %s fields: %s sort: %s for class: %s and replacement: %s " + + "in collection: %s", serializeToJsonSafely(mappedQuery), mappedFields, mappedSort, entityType, - serializeToJsonSafely(replacement), collectionName); + serializeToJsonSafely(replacement), collectionName)); } return executeFindOneInternal( @@ -2898,8 +2900,9 @@ public Publisher doInCollection(MongoCollection collection) if (LOGGER.isDebugEnabled()) { - LOGGER.debug("findOne using query: {} fields: {} in db.collection: {}", serializeToJsonSafely(query), - serializeToJsonSafely(fields.orElseGet(Document::new)), collection.getNamespace().getFullName()); + LOGGER.debug( + String.format("findOne using query: %s fields: %s in db.collection: %s", serializeToJsonSafely(query), + serializeToJsonSafely(fields.orElseGet(Document::new)), collection.getNamespace().getFullName())); } FindPublisher publisher = preparer.initiateFind(collection, col -> col.find(query, Document.class)); diff --git a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/DefaultDbRefResolver.java b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/DefaultDbRefResolver.java index 70fc880cc1..2096142e03 100644 --- a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/DefaultDbRefResolver.java +++ b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/DefaultDbRefResolver.java @@ -22,9 +22,9 @@ import java.util.stream.Collectors; import java.util.stream.Stream; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; import org.bson.Document; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; import org.springframework.dao.InvalidDataAccessApiUsageException; import org.springframework.data.mongodb.MongoDatabaseFactory; @@ -52,7 +52,7 @@ */ public class DefaultDbRefResolver extends DefaultReferenceResolver implements DbRefResolver, ReferenceResolver { - private static final Logger LOGGER = LoggerFactory.getLogger(DefaultDbRefResolver.class); + private static final Log LOGGER = LogFactory.getLog(DefaultDbRefResolver.class); private final MongoDatabaseFactory mongoDbFactory; @@ -130,10 +130,10 @@ public List bulkFetch(List refs) { MongoCollection mongoCollection = getCollection(databaseSource); if (LOGGER.isTraceEnabled()) { - LOGGER.trace("Bulk fetching DBRefs {} from {}.{}.", ids, + LOGGER.trace(String.format("Bulk fetching DBRefs %s from %s.%s.", ids, StringUtils.hasText(databaseSource.getDatabaseName()) ? databaseSource.getDatabaseName() : mongoCollection.getNamespace().getDatabaseName(), - databaseSource.getCollectionName()); + databaseSource.getCollectionName())); } List result = mongoCollection // diff --git a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/LazyLoadingProxyFactory.java b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/LazyLoadingProxyFactory.java index 887ddfe78d..1e0afdd900 100644 --- a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/LazyLoadingProxyFactory.java +++ b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/LazyLoadingProxyFactory.java @@ -25,8 +25,8 @@ import org.aopalliance.intercept.MethodInterceptor; import org.aopalliance.intercept.MethodInvocation; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; import org.springframework.aop.framework.ProxyFactory; import org.springframework.cglib.proxy.Callback; @@ -52,7 +52,7 @@ */ class LazyLoadingProxyFactory { - private static final Logger LOGGER = LoggerFactory.getLogger(LazyLoadingProxyFactory.class); + private static final Log LOGGER = LogFactory.getLog(LazyLoadingProxyFactory.class); private final ObjenesisStd objenesis; @@ -272,16 +272,16 @@ private synchronized Object resolve() { if (resolved) { if (LOGGER.isTraceEnabled()) { - LOGGER.trace("Accessing already resolved lazy loading property {}.{}", - property.getOwner() != null ? property.getOwner().getName() : "unknown", property.getName()); + LOGGER.trace(String.format("Accessing already resolved lazy loading property %s.%s", + property.getOwner() != null ? property.getOwner().getName() : "unknown", property.getName())); } return result; } try { if (LOGGER.isTraceEnabled()) { - LOGGER.trace("Resolving lazy loading property {}.{}", - property.getOwner() != null ? property.getOwner().getName() : "unknown", property.getName()); + LOGGER.trace(String.format("Resolving lazy loading property %s.%s", + property.getOwner() != null ? property.getOwner().getName() : "unknown", property.getName())); } return callback.resolve(property); diff --git a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/MappingMongoConverter.java b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/MappingMongoConverter.java index fa8b4c9643..c8d2c56bad 100644 --- a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/MappingMongoConverter.java +++ b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/MappingMongoConverter.java @@ -29,14 +29,15 @@ import java.util.Set; import java.util.stream.Collectors; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; import org.bson.Document; import org.bson.codecs.Codec; import org.bson.codecs.DecoderContext; import org.bson.conversions.Bson; import org.bson.json.JsonReader; import org.bson.types.ObjectId; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; + import org.springframework.beans.BeansException; import org.springframework.context.ApplicationContext; import org.springframework.context.ApplicationContextAware; @@ -110,7 +111,7 @@ public class MappingMongoConverter extends AbstractMongoConverter implements App public static final ClassTypeInformation BSON = ClassTypeInformation.from(Bson.class); - protected static final Logger LOGGER = LoggerFactory.getLogger(MappingMongoConverter.class); + protected static final Log LOGGER = LogFactory.getLog(MappingMongoConverter.class); protected final MappingContext, MongoPersistentProperty> mappingContext; protected final QueryMapper idMapper; diff --git a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/MongoDatabaseFactoryReferenceLoader.java b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/MongoDatabaseFactoryReferenceLoader.java index d68af1fb5a..eedaa16c49 100644 --- a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/MongoDatabaseFactoryReferenceLoader.java +++ b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/MongoDatabaseFactoryReferenceLoader.java @@ -15,9 +15,10 @@ */ package org.springframework.data.mongodb.core.convert; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; import org.bson.Document; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; + import org.springframework.data.mongodb.MongoDatabaseFactory; import org.springframework.data.mongodb.MongoDatabaseUtils; import org.springframework.data.mongodb.core.convert.ReferenceResolver.ReferenceCollection; @@ -35,7 +36,7 @@ */ public class MongoDatabaseFactoryReferenceLoader implements ReferenceLoader { - private static final Logger LOGGER = LoggerFactory.getLogger(MongoDatabaseFactoryReferenceLoader.class); + private static final Log LOGGER = LogFactory.getLog(MongoDatabaseFactoryReferenceLoader.class); private final MongoDatabaseFactory mongoDbFactory; @@ -55,10 +56,10 @@ public Iterable fetchMany(DocumentReferenceQuery referenceQuery, Refer MongoCollection collection = getCollection(context); if (LOGGER.isTraceEnabled()) { - LOGGER.trace("Bulk fetching {} from {}.{}.", referenceQuery, + LOGGER.trace(String.format("Bulk fetching %s from %s.%s.", referenceQuery, StringUtils.hasText(context.getDatabase()) ? context.getDatabase() : collection.getNamespace().getDatabaseName(), - context.getCollection()); + context.getCollection())); } return referenceQuery.apply(collection); diff --git a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/QueryMapper.java b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/QueryMapper.java index a5d9eabad1..1ae0a15358 100644 --- a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/QueryMapper.java +++ b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/QueryMapper.java @@ -21,12 +21,13 @@ import java.util.regex.Pattern; import java.util.stream.Collectors; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; import org.bson.BsonValue; import org.bson.Document; import org.bson.conversions.Bson; import org.bson.types.ObjectId; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; + import org.springframework.core.convert.ConversionService; import org.springframework.core.convert.converter.Converter; import org.springframework.data.annotation.Reference; @@ -74,7 +75,7 @@ */ public class QueryMapper { - protected static final Logger LOGGER = LoggerFactory.getLogger(QueryMapper.class); + protected static final Log LOGGER = LogFactory.getLog(QueryMapper.class); private static final List DEFAULT_ID_NAMES = Arrays.asList("id", "_id"); private static final Document META_TEXT_SCORE = new Document("$meta", "textScore"); @@ -1224,9 +1225,9 @@ private PersistentPropertyPath getPath(String pathExpre String types = StringUtils.collectionToDelimitedString( path.stream().map(it -> it.getType().getSimpleName()).collect(Collectors.toList()), " -> "); - QueryMapper.LOGGER.info( - "Could not map '{}'. Maybe a fragment in '{}' is considered a simple type. Mapper continues with {}.", - path, types, pathExpression); + QueryMapper.LOGGER.info(String.format( + "Could not map '%s'. Maybe a fragment in '%s' is considered a simple type. Mapper continues with %s.", + path, types, pathExpression)); } return null; } diff --git a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/index/MongoPersistentEntityIndexCreator.java b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/index/MongoPersistentEntityIndexCreator.java index bcfe92e6b1..365990a747 100644 --- a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/index/MongoPersistentEntityIndexCreator.java +++ b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/index/MongoPersistentEntityIndexCreator.java @@ -19,8 +19,9 @@ import java.util.Map; import java.util.concurrent.ConcurrentHashMap; -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.dao.DataIntegrityViolationException; import org.springframework.data.mapping.PersistentEntity; @@ -53,7 +54,7 @@ */ public class MongoPersistentEntityIndexCreator implements ApplicationListener> { - private static final Logger LOGGER = LoggerFactory.getLogger(MongoPersistentEntityIndexCreator.class); + private static final Log LOGGER = LogFactory.getLog(MongoPersistentEntityIndexCreator.class); private final Map, Boolean> classesSeen = new ConcurrentHashMap, Boolean>(); private final IndexOperationsProvider indexOperationsProvider; diff --git a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/index/MongoPersistentEntityIndexResolver.java b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/index/MongoPersistentEntityIndexResolver.java index 5fdb1cbc40..2dc746d785 100644 --- a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/index/MongoPersistentEntityIndexResolver.java +++ b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/index/MongoPersistentEntityIndexResolver.java @@ -27,8 +27,9 @@ import java.util.concurrent.TimeUnit; import java.util.stream.Collectors; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; + import org.springframework.dao.InvalidDataAccessApiUsageException; import org.springframework.data.domain.Sort; import org.springframework.data.mapping.Association; @@ -77,7 +78,7 @@ */ public class MongoPersistentEntityIndexResolver implements IndexResolver { - private static final Logger LOGGER = LoggerFactory.getLogger(MongoPersistentEntityIndexResolver.class); + private static final Log LOGGER = LogFactory.getLog(MongoPersistentEntityIndexResolver.class); private static final SpelExpressionParser PARSER = new SpelExpressionParser(); private final MappingContext, MongoPersistentProperty> mappingContext; diff --git a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/index/ReactiveMongoPersistentEntityIndexCreator.java b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/index/ReactiveMongoPersistentEntityIndexCreator.java index 6a3f6470ee..53f9f4b448 100644 --- a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/index/ReactiveMongoPersistentEntityIndexCreator.java +++ b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/index/ReactiveMongoPersistentEntityIndexCreator.java @@ -23,8 +23,9 @@ import java.util.Map; import java.util.concurrent.ConcurrentHashMap; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; + import org.springframework.dao.DataIntegrityViolationException; import org.springframework.data.mapping.context.MappingContext; import org.springframework.data.mongodb.UncategorizedMongoDbException; @@ -47,7 +48,7 @@ */ public class ReactiveMongoPersistentEntityIndexCreator { - private static final Logger LOGGER = LoggerFactory.getLogger(ReactiveMongoPersistentEntityIndexCreator.class); + private static final Log LOGGER = LogFactory.getLog(ReactiveMongoPersistentEntityIndexCreator.class); private final Map, Boolean> classesSeen = new ConcurrentHashMap, Boolean>(); private final MongoMappingContext mappingContext; diff --git a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/BasicMongoPersistentProperty.java b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/BasicMongoPersistentProperty.java index cf74d696a8..31d9dadfb8 100644 --- a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/BasicMongoPersistentProperty.java +++ b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/BasicMongoPersistentProperty.java @@ -23,9 +23,10 @@ import java.util.List; import java.util.Set; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; import org.bson.types.ObjectId; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; + import org.springframework.data.mapping.Association; import org.springframework.data.mapping.MappingException; import org.springframework.data.mapping.model.AnnotationBasedPersistentProperty; @@ -54,7 +55,7 @@ public class BasicMongoPersistentProperty extends AnnotationBasedPersistentProperty implements MongoPersistentProperty { - private static final Logger LOG = LoggerFactory.getLogger(BasicMongoPersistentProperty.class); + private static final Log LOG = LogFactory.getLog(BasicMongoPersistentProperty.class); public static final String ID_FIELD_NAME = "_id"; private static final String LANGUAGE_FIELD_NAME = "language"; @@ -92,9 +93,9 @@ public BasicMongoPersistentProperty(Property property, MongoPersistentEntity String annotatedName = getAnnotatedFieldName(); if (!ID_FIELD_NAME.equals(annotatedName)) { - LOG.warn( - "Customizing field name for id property '{}.{}' is not allowed! Custom name ('{}') will not be considered!", - owner.getName(), getName(), annotatedName); + LOG.warn(String.format( + "Customizing field name for id property '%s.%s' is not allowed! Custom name ('%s') will not be considered!", + owner.getName(), getName(), annotatedName)); } } } @@ -310,7 +311,7 @@ public boolean isTextScoreProperty() { /** * Obtain the {@link EvaluationContext} for a specific root object. - * + * * @param rootObject can be {@literal null}. * @return never {@literal null}. * @since 3.3 diff --git a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/event/AbstractMongoEventListener.java b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/event/AbstractMongoEventListener.java index e580f1c076..b3be465495 100644 --- a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/event/AbstractMongoEventListener.java +++ b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/event/AbstractMongoEventListener.java @@ -15,8 +15,9 @@ */ package org.springframework.data.mongodb.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; @@ -30,7 +31,7 @@ */ public abstract class AbstractMongoEventListener implements ApplicationListener> { - private static final Logger LOG = LoggerFactory.getLogger(AbstractMongoEventListener.class); + private static final Log LOG = LogFactory.getLog(AbstractMongoEventListener.class); private final Class domainClass; /** @@ -103,7 +104,7 @@ public void onApplicationEvent(MongoMappingEvent event) { public void onBeforeConvert(BeforeConvertEvent event) { if (LOG.isDebugEnabled()) { - LOG.debug("onBeforeConvert({})", event.getSource()); + LOG.debug(String.format("onBeforeConvert(%s)", event.getSource())); } } @@ -116,7 +117,7 @@ public void onBeforeConvert(BeforeConvertEvent event) { public void onBeforeSave(BeforeSaveEvent event) { if (LOG.isDebugEnabled()) { - LOG.debug("onBeforeSave({}, {})", event.getSource(), event.getDocument()); + LOG.debug(String.format("onBeforeSave(%s, %s)", event.getSource(), event.getDocument())); } } @@ -129,7 +130,7 @@ public void onBeforeSave(BeforeSaveEvent event) { public void onAfterSave(AfterSaveEvent event) { if (LOG.isDebugEnabled()) { - LOG.debug("onAfterSave({}, {})", event.getSource(), event.getDocument()); + LOG.debug(String.format("onAfterSave(%s, %s)", event.getSource(), event.getDocument())); } } @@ -142,7 +143,7 @@ public void onAfterSave(AfterSaveEvent event) { public void onAfterLoad(AfterLoadEvent event) { if (LOG.isDebugEnabled()) { - LOG.debug("onAfterLoad({})", event.getDocument()); + LOG.debug(String.format("onAfterLoad(%s)", event.getDocument())); } } @@ -155,7 +156,7 @@ public void onAfterLoad(AfterLoadEvent event) { public void onAfterConvert(AfterConvertEvent event) { if (LOG.isDebugEnabled()) { - LOG.debug("onAfterConvert({}, {})", event.getDocument(), event.getSource()); + LOG.debug(String.format("onAfterConvert(%s, %s)", event.getDocument(), event.getSource())); } } @@ -168,7 +169,7 @@ public void onAfterConvert(AfterConvertEvent event) { public void onAfterDelete(AfterDeleteEvent event) { if (LOG.isDebugEnabled()) { - LOG.debug("onAfterDelete({})", event.getDocument()); + LOG.debug(String.format("onAfterDelete(%s)", event.getDocument())); } } @@ -181,7 +182,7 @@ public void onAfterDelete(AfterDeleteEvent event) { public void onBeforeDelete(BeforeDeleteEvent event) { if (LOG.isDebugEnabled()) { - LOG.debug("onBeforeDelete({})", event.getDocument()); + LOG.debug(String.format("onBeforeDelete(%s)", event.getDocument())); } } } diff --git a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/event/LoggingEventListener.java b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/event/LoggingEventListener.java index 2f0d4f5247..b86bb7f394 100644 --- a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/event/LoggingEventListener.java +++ b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/event/LoggingEventListener.java @@ -17,8 +17,9 @@ import static org.springframework.data.mongodb.core.query.SerializationUtils.*; -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; /** @@ -31,7 +32,7 @@ */ public class LoggingEventListener extends AbstractMongoEventListener { - private static final Logger LOGGER = LoggerFactory.getLogger(LoggingEventListener.class); + private static final Log LOGGER = LogFactory.getLog(LoggingEventListener.class); /* * (non-Javadoc) @@ -39,7 +40,7 @@ public class LoggingEventListener extends AbstractMongoEventListener { */ @Override public void onBeforeConvert(BeforeConvertEvent event) { - LOGGER.info("onBeforeConvert: {}", event.getSource()); + LOGGER.info(String.format("onBeforeConvert: %s", event.getSource())); } /* @@ -48,7 +49,7 @@ public void onBeforeConvert(BeforeConvertEvent event) { */ @Override public void onBeforeSave(BeforeSaveEvent event) { - LOGGER.info("onBeforeSave: {}, {}", event.getSource(), serializeToJsonSafely(event.getDocument())); + LOGGER.info(String.format("onBeforeSave: %s, %s", event.getSource(), serializeToJsonSafely(event.getDocument()))); } /* @@ -57,7 +58,7 @@ public void onBeforeSave(BeforeSaveEvent event) { */ @Override public void onAfterSave(AfterSaveEvent event) { - LOGGER.info("onAfterSave: {}, {}", event.getSource(), serializeToJsonSafely(event.getDocument())); + LOGGER.info(String.format("onAfterSave: %s, %s", event.getSource(), serializeToJsonSafely(event.getDocument()))); } /* @@ -66,7 +67,7 @@ public void onAfterSave(AfterSaveEvent event) { */ @Override public void onAfterLoad(AfterLoadEvent event) { - LOGGER.info("onAfterLoad: {}", serializeToJsonSafely(event.getDocument())); + LOGGER.info(String.format("onAfterLoad: %s", serializeToJsonSafely(event.getDocument()))); } /* @@ -75,7 +76,7 @@ public void onAfterLoad(AfterLoadEvent event) { */ @Override public void onAfterConvert(AfterConvertEvent event) { - LOGGER.info("onAfterConvert: {}, {}", serializeToJsonSafely(event.getDocument()), event.getSource()); + LOGGER.info(String.format("onAfterConvert: %s, %s", serializeToJsonSafely(event.getDocument()), event.getSource())); } /* @@ -84,7 +85,7 @@ public void onAfterConvert(AfterConvertEvent event) { */ @Override public void onAfterDelete(AfterDeleteEvent event) { - LOGGER.info("onAfterDelete: {}", serializeToJsonSafely(event.getDocument())); + LOGGER.info(String.format("onAfterDelete: %s", serializeToJsonSafely(event.getDocument()))); } /* @@ -93,6 +94,6 @@ public void onAfterDelete(AfterDeleteEvent event) { */ @Override public void onBeforeDelete(BeforeDeleteEvent event) { - LOGGER.info("onBeforeDelete: {}", serializeToJsonSafely(event.getDocument())); + LOGGER.info(String.format("onBeforeDelete: %s", serializeToJsonSafely(event.getDocument()))); } } diff --git a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/event/ValidatingMongoEventListener.java b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/event/ValidatingMongoEventListener.java index d62831d6ab..9f93a02741 100644 --- a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/event/ValidatingMongoEventListener.java +++ b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/event/ValidatingMongoEventListener.java @@ -20,8 +20,9 @@ import javax.validation.ConstraintViolationException; import javax.validation.Validator; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; + import org.springframework.util.Assert; /** @@ -34,7 +35,7 @@ */ public class ValidatingMongoEventListener extends AbstractMongoEventListener { - private static final Logger LOG = LoggerFactory.getLogger(ValidatingMongoEventListener.class); + private static final Log LOG = LogFactory.getLog(ValidatingMongoEventListener.class); private final Validator validator; @@ -57,12 +58,16 @@ public ValidatingMongoEventListener(Validator validator) { @Override public void onBeforeSave(BeforeSaveEvent event) { - LOG.debug("Validating object: {}", event.getSource()); + if (LOG.isDebugEnabled()) { + LOG.debug(String.format("Validating object: {}", event.getSource())); + } Set violations = validator.validate(event.getSource()); if (!violations.isEmpty()) { - LOG.info("During object: {} validation violations found: {}", event.getSource(), violations); + if (LOG.isDebugEnabled()) { + LOG.info(String.format("During object: {} validation violations found: {}", event.getSource(), violations)); + } throw new ConstraintViolationException(violations); } } diff --git a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/repository/cdi/MongoRepositoryExtension.java b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/repository/cdi/MongoRepositoryExtension.java index 7bb9dfa237..591a0893a3 100644 --- a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/repository/cdi/MongoRepositoryExtension.java +++ b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/repository/cdi/MongoRepositoryExtension.java @@ -31,8 +31,9 @@ import javax.enterprise.inject.spi.BeanManager; import javax.enterprise.inject.spi.ProcessBean; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; + import org.springframework.data.mongodb.core.MongoOperations; import org.springframework.data.repository.cdi.CdiRepositoryBean; import org.springframework.data.repository.cdi.CdiRepositoryExtensionSupport; @@ -45,7 +46,7 @@ */ public class MongoRepositoryExtension extends CdiRepositoryExtensionSupport { - private static final Logger LOG = LoggerFactory.getLogger(MongoRepositoryExtension.class); + private static final Log LOG = LogFactory.getLog(MongoRepositoryExtension.class); private final Map, Bean> mongoOperations = new HashMap, Bean>(); diff --git a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/repository/query/MongoQueryCreator.java b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/repository/query/MongoQueryCreator.java index eae562e4b1..dfaa7fde6d 100644 --- a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/repository/query/MongoQueryCreator.java +++ b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/repository/query/MongoQueryCreator.java @@ -23,8 +23,9 @@ import java.util.Optional; import java.util.regex.Pattern; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; + import org.springframework.data.domain.Range; import org.springframework.data.domain.Range.Bound; import org.springframework.data.domain.Sort; @@ -65,7 +66,7 @@ */ class MongoQueryCreator extends AbstractQueryCreator { - private static final Logger LOG = LoggerFactory.getLogger(MongoQueryCreator.class); + private static final Log LOG = LogFactory.getLog(MongoQueryCreator.class); private final MongoParameterAccessor accessor; private final MappingContext context; diff --git a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/repository/query/ReactiveStringBasedMongoQuery.java b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/repository/query/ReactiveStringBasedMongoQuery.java index 2ccd20e441..97c2f551ed 100644 --- a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/repository/query/ReactiveStringBasedMongoQuery.java +++ b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/repository/query/ReactiveStringBasedMongoQuery.java @@ -17,9 +17,10 @@ import reactor.core.publisher.Mono; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; import org.bson.Document; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; + import org.springframework.data.mongodb.core.MongoOperations; import org.springframework.data.mongodb.core.ReactiveMongoOperations; import org.springframework.data.mongodb.core.query.BasicQuery; @@ -43,13 +44,12 @@ public class ReactiveStringBasedMongoQuery extends AbstractReactiveMongoQuery { private static final String COUNT_EXISTS_AND_DELETE = "Manually defined query for %s cannot be a count and exists or delete query at the same time!"; - private static final Logger LOG = LoggerFactory.getLogger(ReactiveStringBasedMongoQuery.class); + private static final Log LOG = LogFactory.getLog(ReactiveStringBasedMongoQuery.class); private final String query; private final String fieldSpec; private final ExpressionParser expressionParser; - private final ReactiveQueryMethodEvaluationContextProvider evaluationContextProvider; private final boolean isCountQuery; private final boolean isExistsQuery; @@ -90,7 +90,6 @@ public ReactiveStringBasedMongoQuery(String query, ReactiveMongoQueryMethod meth this.query = query; this.expressionParser = expressionParser; - this.evaluationContextProvider = evaluationContextProvider; this.fieldSpec = method.getFieldSpecification(); if (method.hasAnnotatedQuery()) { diff --git a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/repository/query/StringBasedMongoQuery.java b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/repository/query/StringBasedMongoQuery.java index fb721a3be4..abeef15ecd 100644 --- a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/repository/query/StringBasedMongoQuery.java +++ b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/repository/query/StringBasedMongoQuery.java @@ -15,9 +15,10 @@ */ package org.springframework.data.mongodb.repository.query; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; import org.bson.Document; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; + import org.springframework.data.mapping.model.SpELExpressionEvaluator; import org.springframework.data.mongodb.core.MongoOperations; import org.springframework.data.mongodb.core.query.BasicQuery; @@ -41,13 +42,12 @@ public class StringBasedMongoQuery extends AbstractMongoQuery { private static final String COUNT_EXISTS_AND_DELETE = "Manually defined query for %s cannot be a count and exists or delete query at the same time!"; - private static final Logger LOG = LoggerFactory.getLogger(StringBasedMongoQuery.class); + private static final Log LOG = LogFactory.getLog(StringBasedMongoQuery.class); private final String query; private final String fieldSpec; private final ExpressionParser expressionParser; - private final QueryMethodEvaluationContextProvider evaluationContextProvider; private final boolean isCountQuery; private final boolean isExistsQuery; @@ -86,7 +86,6 @@ public StringBasedMongoQuery(String query, MongoQueryMethod method, MongoOperati this.query = query; this.expressionParser = expressionParser; - this.evaluationContextProvider = evaluationContextProvider; this.fieldSpec = method.getFieldSpecification(); if (method.hasAnnotatedQuery()) { diff --git a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/repository/support/IndexEnsuringQueryCreationListener.java b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/repository/support/IndexEnsuringQueryCreationListener.java index b1c3de6509..43f66810aa 100644 --- a/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/repository/support/IndexEnsuringQueryCreationListener.java +++ b/spring-data-mongodb/src/main/java/org/springframework/data/mongodb/repository/support/IndexEnsuringQueryCreationListener.java @@ -20,8 +20,9 @@ import java.util.HashSet; import java.util.Set; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; + import org.springframework.core.annotation.AnnotatedElementUtils; import org.springframework.data.domain.Sort; import org.springframework.data.domain.Sort.Direction; @@ -54,7 +55,7 @@ class IndexEnsuringQueryCreationListener implements QueryCreationListener { private static final Set GEOSPATIAL_TYPES = new HashSet(Arrays.asList(Type.NEAR, Type.WITHIN)); - private static final Logger LOG = LoggerFactory.getLogger(IndexEnsuringQueryCreationListener.class); + private static final Log LOG = LogFactory.getLog(IndexEnsuringQueryCreationListener.class); private final IndexOperationsProvider indexOperationsProvider; diff --git a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/core/PersonExample.java b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/core/PersonExample.java index d935a85d5c..69847840f5 100644 --- a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/core/PersonExample.java +++ b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/core/PersonExample.java @@ -17,8 +17,9 @@ import java.util.List; -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.annotation.AnnotationConfigApplicationContext; import org.springframework.context.support.AbstractApplicationContext; @@ -29,7 +30,7 @@ */ public class PersonExample { - private static final Logger LOGGER = LoggerFactory.getLogger(PersonExample.class); + private static final Log LOGGER = LogFactory.getLog(PersonExample.class); @Autowired private MongoOperations mongoOps; diff --git a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/core/aggregation/AggregationTests.java b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/core/aggregation/AggregationTests.java index 2cf1b3fbb5..991523108a 100644 --- a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/core/aggregation/AggregationTests.java +++ b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/core/aggregation/AggregationTests.java @@ -33,6 +33,8 @@ import java.util.List; import java.util.Scanner; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; import org.assertj.core.data.Offset; import org.bson.Document; import org.joda.time.DateTime; @@ -42,8 +44,6 @@ import org.junit.jupiter.api.BeforeEach; import org.junit.jupiter.api.Test; import org.junit.jupiter.api.extension.ExtendWith; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; import org.springframework.core.io.ClassPathResource; import org.springframework.dao.DataAccessException; import org.springframework.data.annotation.Id; @@ -94,7 +94,7 @@ public class AggregationTests { private static final String INPUT_COLLECTION = "aggregation_test_collection"; - private static final Logger LOGGER = LoggerFactory.getLogger(AggregationTests.class); + private static final Log LOGGER = LogFactory.getLog(AggregationTests.class); private static boolean initialized = false; diff --git a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/core/mapping/event/LoggingEventListenerTests.java b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/core/mapping/event/LoggingEventListenerTests.java index 8ac79c783c..6eeb20cfec 100644 --- a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/core/mapping/event/LoggingEventListenerTests.java +++ b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/core/mapping/event/LoggingEventListenerTests.java @@ -42,7 +42,7 @@ public class LoggingEventListenerTests { @BeforeEach public void setUp() { - appender = new ListAppender(); + appender = new ListAppender<>(); // set log level for LoggingEventListener to "info" and set up an appender capturing events. logger = (ch.qos.logback.classic.Logger) LoggerFactory.getLogger(LoggingEventListener.class); diff --git a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/test/util/CleanMongoDB.java b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/test/util/CleanMongoDB.java index 1dd3856049..610363c93d 100644 --- a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/test/util/CleanMongoDB.java +++ b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/test/util/CleanMongoDB.java @@ -23,12 +23,13 @@ import java.util.LinkedHashSet; import java.util.Set; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; import org.bson.Document; import org.junit.rules.TestRule; import org.junit.runner.Description; import org.junit.runners.model.Statement; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; + import org.springframework.util.CollectionUtils; import org.springframework.util.StringUtils; @@ -48,7 +49,7 @@ */ public class CleanMongoDB implements TestRule { - private static final Logger LOGGER = LoggerFactory.getLogger(CleanMongoDB.class); + private static final Log LOGGER = LogFactory.getLog(CleanMongoDB.class); /** * Defines contents of MongoDB. @@ -288,7 +289,7 @@ private boolean dropDbIfRequired(String dbName) { } client.getDatabase(dbName).drop(); - LOGGER.debug("Dropping DB '{}'. ", dbName); + LOGGER.debug(String.format("Dropping DB '%s'. ", dbName)); return true; } @@ -305,10 +306,11 @@ private void dropCollectionsOrIndexIfRequried(MongoDatabase db, Collection type, ExtensionContext extensionContext private ReactiveClientHolder reactiveClient(boolean replSet) { - LOGGER.debug("Creating new reactive {}client.", replSet ? "replica set " : ""); + LOGGER.debug(String.format("Creating new reactive %sclient.", replSet ? "replica set " : "")); return new ReactiveClientHolder(replSet ? MongoTestUtils.reactiveReplSetClient() : MongoTestUtils.reactiveClient()); } private SyncClientHolder syncClient(boolean replSet) { - LOGGER.debug("Creating new sync {}client.", replSet ? "replica set " : ""); + LOGGER.debug(String.format("Creating new sync %sclient.", replSet ? "replica set " : "")); return new SyncClientHolder(replSet ? MongoTestUtils.replSetClient() : MongoTestUtils.client()); } @@ -135,9 +136,9 @@ private void assertValidFieldCandidate(Field field) { private void assertSupportedType(String target, Class type) { if (type != com.mongodb.client.MongoClient.class && type != com.mongodb.reactivestreams.client.MongoClient.class) { - throw new ExtensionConfigurationException("Can only resolve @MongoClient " + target + " of type " - + com.mongodb.client.MongoClient.class.getName() + " or " - + com.mongodb.reactivestreams.client.MongoClient.class.getName() + " but was: " + type.getName()); + throw new ExtensionConfigurationException(String.format( + "Can only resolve @MongoClient %s of type %s or %s but was: %s", target, MongoClient.class.getName(), + com.mongodb.reactivestreams.client.MongoClient.class.getName(), type.getName())); } }