stefan-egli commented on code in PR #1402:
URL: https://github.com/apache/jackrabbit-oak/pull/1402#discussion_r1552185203
##########
oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/VersionGarbageCollector.java:
##########
@@ -1048,8 +1061,9 @@ public void collectGarbage(final NodeDocument doc, final
GCPhases phases) {
updateOpList.add(op);
}
}
- if (log.isDebugEnabled()) {
- log.debug("UpdateOp for {} is {}", doc.getId(), op);
+ if (DETAILED_GC_LOG.isDebugEnabled() && op.hasChanges()) {
+ // only log in case of changes & debug level enabled
+ DETAILED_GC_LOG.debug("UpdateOp for {} is {}", doc.getId(),
op);
Review Comment:
Not sure we should have the entire UpdateOp in the audit log - it can be
very noisy. Also not sure if logging the property names is fine here... (we
could move it to log.trace, perhaps, not sure..)
--
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: dev-unsubscr...@jackrabbit.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
