nfsantos commented on code in PR #2102:
URL: https://github.com/apache/jackrabbit-oak/pull/2102#discussion_r1963957199


##########
oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentNodeStore.java:
##########
@@ -3429,7 +3429,7 @@ private String diffImpl(AbstractDocumentNodeState from, 
AbstractDocumentNodeStat
                 // see OAK-6016 and OAK-6011
                 if (e instanceof IllegalStateException &&
                         "Root document does not have a lastRev entry for local 
clusterId 0".equals(e.getMessage())) {
-                    LOG.warn("diffJournalChildren failed with " +
+                    LOG.debug("diffJournalChildren failed with " +

Review Comment:
   It is more efficient to use the parametrized version of this method (with {} 
placeholders). As this log will rarely be printed, it is wasteful to create the 
String that is passed as an argument.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to