You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-commits@jackrabbit.apache.org by st...@apache.org on 2023/12/14 17:19:29 UTC

(jackrabbit-oak) branch OAK-10199-jenkinstroubles updated: OAK-10199 related : in limit exceeded case detailed GC could never have been started

This is an automated email from the ASF dual-hosted git repository.

stefanegli pushed a commit to branch OAK-10199-jenkinstroubles
in repository https://gitbox.apache.org/repos/asf/jackrabbit-oak.git


The following commit(s) were added to refs/heads/OAK-10199-jenkinstroubles by this push:
     new e07ef01e6a OAK-10199 related : in limit exceeded case detailed GC could never have been started
e07ef01e6a is described below

commit e07ef01e6a9e210cdf727b56cb47cf986bab8663
Author: Stefan Egli <st...@apache.org>
AuthorDate: Thu Dec 14 18:19:19 2023 +0100

    OAK-10199 related : in limit exceeded case detailed GC could never have been started
---
 .../jackrabbit/oak/plugins/document/VersionGarbageCollector.java      | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

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 6ca5c468c1..c54d8f1b35 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
@@ -653,7 +653,9 @@ public class VersionGarbageCollector {
             monitor.info("Revision garbage collection finished in {}. {}",
                     TimeDurationFormatter.forLogging().format(phases.elapsed.elapsed(MICROSECONDS), MICROSECONDS), stats);
             if (detailedGCEnabled) {
-                stats.detailedGCActive.stop();
+                if (stats.detailedGCActive.isRunning()) {
+                    stats.detailedGCActive.stop();
+                } // probably limitExceeded == true case
             }
             stats.active.stop();
             return stats;