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 14:36:07 UTC

(jackrabbit-oak) branch OAK-10584 created (now b4f1aa6658)

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

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


      at b4f1aa6658 OAK-10584 : catch exception in getOldestRevisionToKeep when in read-only mode

This branch includes the following new commits:

     new b4f1aa6658 OAK-10584 : catch exception in getOldestRevisionToKeep when in read-only mode

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-10584 : catch exception in getOldestRevisionToKeep when in read-only mode

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

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

commit b4f1aa66588c4e7ed3fc58cf04f76be7e76116ed
Author: stefan-egli <st...@apache.org>
AuthorDate: Thu Dec 14 15:36:01 2023 +0100

    OAK-10584 : catch exception in getOldestRevisionToKeep when in read-only mode
---
 .../org/apache/jackrabbit/oak/plugins/document/Checkpoints.java  | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/Checkpoints.java b/oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/Checkpoints.java
index 50c61f6da3..4624ea4835 100644
--- a/oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/Checkpoints.java
+++ b/oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/Checkpoints.java
@@ -146,8 +146,13 @@ class Checkpoints {
         }
 
         if (op.hasChanges()) {
-            store.findAndUpdate(Collection.SETTINGS, op);
-            LOG.debug("Purged {} expired checkpoints", op.getChanges().size());
+        	try {
+        		store.findAndUpdate(Collection.SETTINGS, op);
+        		LOG.debug("Purged {} expired checkpoints", op.getChanges().size());
+        	} catch (UnsupportedOperationException uoe) {
+        		LOG.info("getOldestRevisionToKeep : could not clean up expired checkpoints"
+        				+ " due to excepttion : " + uoe, uoe);
+        	}
         }
 
         return lastAliveRevision;