From f2f81e895bbb346a10aef499e9112156ef42e12e Mon Sep 17 00:00:00 2001 From: Mark Paluch Date: Wed, 10 Apr 2024 16:08:02 +0200 Subject: [PATCH] Polishing. Consistent log field ordering, serialize sort safely. See #4686 Original pull request: #4679 --- .../data/mongodb/core/MongoTemplate.java | 15 +++++++++------ .../data/mongodb/core/ReactiveMongoTemplate.java | 7 ++++--- 2 files changed, 13 insertions(+), 9 deletions(-) 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 ce21c72c2..ba59d8605 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 @@ -564,8 +564,8 @@ public class MongoTemplate Document fieldsObject = query.getFieldsObject(); if (LOGGER.isDebugEnabled()) { - LOGGER.debug(String.format("Executing query: %s sort: %s fields: %s in collection: %s", - serializeToJsonSafely(queryObject), sortObject, fieldsObject, collectionName)); + LOGGER.debug(String.format("Executing query: %s fields: %s sort: %s in collection: %s", + serializeToJsonSafely(queryObject), fieldsObject, serializeToJsonSafely(sortObject), collectionName)); } this.executeQueryInternal(new FindCallback(createDelegate(query), queryObject, fieldsObject, null), @@ -2599,7 +2599,8 @@ public class MongoTemplate if (LOGGER.isDebugEnabled()) { LOGGER.debug(String.format("find using query: %s fields: %s sort: %s for class: %s in collection: %s", - serializeToJsonSafely(mappedQuery), mappedSort, mappedFields, entityClass, collectionName)); + serializeToJsonSafely(mappedQuery), mappedFields, serializeToJsonSafely(mappedSort), entityClass, + collectionName)); } return executeFindMultiInternal(new FindCallback(collectionPreparer, mappedQuery, mappedFields, null), @@ -2625,7 +2626,8 @@ public class MongoTemplate if (LOGGER.isDebugEnabled()) { LOGGER.debug(String.format("find using query: %s fields: %s sort: %s for class: %s in collection: %s", - serializeToJsonSafely(mappedQuery), mappedSort, mappedFields, sourceClass, collectionName)); + serializeToJsonSafely(mappedQuery), mappedFields, serializeToJsonSafely(mappedSort), sourceClass, + collectionName)); } return executeFindMultiInternal(new FindCallback(collectionPreparer, mappedQuery, mappedFields, null), preparer, @@ -2707,7 +2709,7 @@ public class MongoTemplate if (LOGGER.isDebugEnabled()) { LOGGER.debug(String.format("findAndRemove using query: %s fields: %s sort: %s for class: %s in collection: %s", - serializeToJsonSafely(query), fields, sort, entityClass, collectionName)); + serializeToJsonSafely(query), fields, serializeToJsonSafely(sort), entityClass, collectionName)); } MongoPersistentEntity entity = mappingContext.getPersistentEntity(entityClass); @@ -2738,7 +2740,8 @@ public class MongoTemplate if (LOGGER.isDebugEnabled()) { 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), + serializeToJsonSafely(mappedQuery), fields, serializeToJsonSafely(sort), entityClass, + serializeToJsonSafely(mappedUpdate), collectionName)); } 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 146bd0cb9..99dfea85e 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 @@ -2458,7 +2458,7 @@ public class ReactiveMongoTemplate implements ReactiveMongoOperations, Applicati if (LOGGER.isDebugEnabled()) { LOGGER.debug(String.format("findAndRemove using query: %s fields: %s sort: %s for class: %s in collection: %s", - serializeToJsonSafely(query), fields, sort, entityClass, collectionName)); + serializeToJsonSafely(query), fields, serializeToJsonSafely(sort), entityClass, collectionName)); } MongoPersistentEntity entity = mappingContext.getPersistentEntity(entityClass); @@ -2485,7 +2485,8 @@ public class ReactiveMongoTemplate implements ReactiveMongoOperations, Applicati if (LOGGER.isDebugEnabled()) { 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), + serializeToJsonSafely(mappedQuery), fields, serializeToJsonSafely(sort), entityClass, + serializeToJsonSafely(mappedUpdate), collectionName)); } @@ -2552,7 +2553,7 @@ public class ReactiveMongoTemplate implements ReactiveMongoOperations, Applicati 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(mappedQuery), mappedFields, serializeToJsonSafely(mappedSort), entityType, serializeToJsonSafely(replacement), collectionName)); }