From 307e8fce11e5ef467c8b26bb8c39ae1440fedd14 Mon Sep 17 00:00:00 2001 From: "evgeny.bovykin" Date: Tue, 10 May 2022 13:43:54 +0200 Subject: [PATCH] Fix debug logging in MergingPersistenceUnitManager. A String.format token was missing, so `mappingFileName` was printed instead of a `persistenceUnitName`, and `persistenceUnitName` wasn't actually printed at all. See #2526. --- .../data/jpa/support/MergingPersistenceUnitManager.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/main/java/org/springframework/data/jpa/support/MergingPersistenceUnitManager.java b/src/main/java/org/springframework/data/jpa/support/MergingPersistenceUnitManager.java index 294560f74..fed263bad 100644 --- a/src/main/java/org/springframework/data/jpa/support/MergingPersistenceUnitManager.java +++ b/src/main/java/org/springframework/data/jpa/support/MergingPersistenceUnitManager.java @@ -82,7 +82,9 @@ public class MergingPersistenceUnitManager extends DefaultPersistenceUnitManager for (String mappingFileName : oldPui.getMappingFileNames()) { if (!pui.getMappingFileNames().contains(mappingFileName)) { - LOG.debug("Adding mapping file to persistence unit {}.", mappingFileName, persistenceUnitName); + if (LOG.isDebugEnabled()) { + LOG.debug(String.format("Adding mapping file %s to persistence unit %s.", mappingFileName, persistenceUnitName)); + } pui.addMappingFileName(mappingFileName); } }