stefan-egli commented on code in PR #1402:
URL: https://github.com/apache/jackrabbit-oak/pull/1402#discussion_r1552193207
##########
oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/VersionGarbageCollector.java:
##########
@@ -1613,8 +1636,8 @@ private int removeUnusedPropertyEntries(NodeDocument doc,
// could be due to node not existing or current value being in
a split
// doc - while the former is unexpected, the latter might
happen.
// in both cases let's skip this property
- if (log.isDebugEnabled()) {
- log.debug("removeUnusedPropertyEntries : no visible
revision for property {} in doc {}",
+ if (DETAILED_GC_LOG.isDebugEnabled()) {
+ DETAILED_GC_LOG.debug("removeUnusedPropertyEntries : no
visible revision for property {} in doc {}",
Review Comment:
I guess we should decide how we treat property names - but this one is
similar to the others and I think it should be log.trace
--
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
