This is an automated email from the ASF dual-hosted git repository. stefanegli pushed a commit to branch trunk in repository https://gitbox.apache.org/repos/asf/jackrabbit-oak.git
The following commit(s) were added to refs/heads/trunk by this push: new 011a4e1320 OAK-11251: Improve warning in collectFullGC to reduce its noisiness. (#1844) 011a4e1320 is described below commit 011a4e13204c6fadc4d6ad211d4e11f0b8d47647 Author: José Andrés Cordero Benítez <josco...@users.noreply.github.com> AuthorDate: Wed Nov 6 12:15:29 2024 +0100 OAK-11251: Improve warning in collectFullGC to reduce its noisiness. (#1844) --- .../jackrabbit/oak/plugins/document/VersionGarbageCollector.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/VersionGarbageCollector.java b/oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/VersionGarbageCollector.java index 926f7c099e..ad9cca9bea 100644 --- a/oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/VersionGarbageCollector.java +++ b/oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/VersionGarbageCollector.java @@ -937,9 +937,9 @@ public class VersionGarbageCollector { final Long modified = lastDoc.getModified(); if (modified == null) { monitor.warn("collectFullGC : document has no _modified property : {}", doc.getId()); - } else if (SECONDS.toMillis(modified) < fromModifiedMs) { - monitor.warn("collectFullGC : document has older _modified than query boundary : {} (from: {}, to: {})", - modified, timestampToString(fromModifiedMs), timestampToString(toModifiedMs)); + } else if (modified < MILLISECONDS.toSeconds(fromModifiedMs)) { + monitor.warn("collectFullGC : document has older _modified than query boundary : {} (from: {} [{}], to: {} [{}])", + modified, fromModifiedMs, timestampToString(fromModifiedMs), toModifiedMs, timestampToString(toModifiedMs)); } } // now remove the garbage in one go, if any