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 ad...@apache.org on 2022/07/19 15:05:35 UTC

[jackrabbit-oak] branch issues/OAK-9847 created (now 1118295784)

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

adulceanu pushed a change to branch issues/OAK-9847
in repository https://gitbox.apache.org/repos/asf/jackrabbit-oak.git


      at 1118295784 OAK-9847 - Offline compaction cleanup failures for migrated AzureSegmentStores with missing graph files

This branch includes the following new commits:

     new 1118295784 OAK-9847 - Offline compaction cleanup failures for migrated AzureSegmentStores with missing graph files

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.

[jackrabbit-oak] 01/01: OAK-9847 - Offline compaction cleanup failures for migrated AzureSegmentStores with missing graph files

Posted by ad...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

adulceanu pushed a commit to branch issues/OAK-9847
in repository https://gitbox.apache.org/repos/asf/jackrabbit-oak.git

commit 1118295784a6f0cc7cb2d9bf381878b97922d987
Author: dulceanu <an...@gmail.com>
AuthorDate: Tue Jul 19 18:04:47 2022 +0300

    OAK-9847 - Offline compaction cleanup failures for migrated AzureSegmentStores with missing graph files
---
 .../org/apache/jackrabbit/oak/segment/file/DefaultCleanupStrategy.java | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/DefaultCleanupStrategy.java b/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/DefaultCleanupStrategy.java
index 527bd694da..92d0dbea4c 100644
--- a/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/DefaultCleanupStrategy.java
+++ b/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/DefaultCleanupStrategy.java
@@ -103,7 +103,8 @@ class DefaultCleanupStrategy implements CleanupStrategy {
 
             @Override
             public boolean shouldReclaim(UUID id, GCGeneration generation, boolean referenced) {
-                return isUnreferencedBulkSegment(id, referenced) || isOldDataSegment(id, generation);
+                return isUnreferencedBulkSegment(id, referenced) || isOldDataSegment(id, generation)
+                        || generation.getFullGeneration() > getGcGeneration(context).getFullGeneration();
             }
 
             @Override