This is an automated email from the ASF dual-hosted git repository. stefanegli pushed a commit to branch OAK-11477 in repository https://gitbox.apache.org/repos/asf/jackrabbit-oak.git
commit 9393f872eda28f36a30093020a68d5b6fd60a40e Author: stefan-egli <[email protected]> AuthorDate: Mon Feb 17 15:06:04 2025 +0100 OAK-11477 : dont fail if lease couldnt be renewed in crash simulation --- .../document/BaseDocumentDiscoveryLiteServiceTest.java | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/BaseDocumentDiscoveryLiteServiceTest.java b/oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/BaseDocumentDiscoveryLiteServiceTest.java index 539f27aac9..18e35978e8 100644 --- a/oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/BaseDocumentDiscoveryLiteServiceTest.java +++ b/oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/BaseDocumentDiscoveryLiteServiceTest.java @@ -283,10 +283,13 @@ public abstract class BaseDocumentDiscoveryLiteServiceTest { logger.info("crash: stopped lastrev thread, now setting lease to end within 1 sec"); boolean renewed = setLeaseTime(1000 /* 1 sec */, 10 /*10ms*/); - if (!renewed) { - logger.info("halt"); - fail("did not renew clusterid lease"); - } + logger.info("renewed : {}", renewed); + // OAK-11477: lease could have been update by lease update thread while we wanted to do that + // in which case testing for !renewed is not very useful here + //if (!renewed) { + // logger.info("halt"); + // fail("did not renew clusterid lease"); + //} logger.info("crash: now stopping background read/update"); stopAllBackgroundThreads();
