Skip to content

OAK-12437: gate ASYNC maintenance and entry metadata together - #3190

Open
rishabhdaim wants to merge 1 commit into
OAK-12425from
issue/OAK-12437
Open

OAK-12437: gate ASYNC maintenance and entry metadata together#3190
rishabhdaim wants to merge 1 commit into
OAK-12425from
issue/OAK-12437

OAK-12437: add opt-in ASYNC maintenance with entry metadata

98d878d
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Oct 8, 2026 in 2m 52s

Quality Gate failed

Failed conditions
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Annotations

Check warning on line 84 in oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/persistentCache/CacheEntry.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a thread-safe type; adding "volatile" is not enough to make this field thread-safe. 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AaEbbV0PzeCxtYUacnxg&open=AaEbbV0PzeCxtYUacnxg&pullRequest=3190

Check failure on line 57 in oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/persistentCache/CacheEntry.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove usage of generic wildcard type. 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AaEbbV0PzeCxtYUacnxf&open=AaEbbV0PzeCxtYUacnxf&pullRequest=3190

Check warning on line 70 in oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/persistentCache/async/CacheWriteQueueTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of thrown exception 'java.lang.Exception', as it cannot be thrown from method's body. 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AaEbbV7DzeCxtYUacnxj&open=AaEbbV7DzeCxtYUacnxj&pullRequest=3190

Check warning on line 190 in oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/persistentCache/AsyncMetadataQueueTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of thrown exception 'java.lang.Exception', as it cannot be thrown from method's body. 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AaEbbV7ezeCxtYUacnxp&open=AaEbbV7ezeCxtYUacnxp&pullRequest=3190

Check warning on line 119 in oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/persistentCache/async/CacheWriteQueueTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor the code of the lambda to have only one invocation possibly throwing a runtime exception. 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AaEbbV7DzeCxtYUacnxm&open=AaEbbV7DzeCxtYUacnxm&pullRequest=3190

Check warning on line 341 in oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/DocumentCacheMaintenanceTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename "previousCaffeine" which hides the field declared at line 48. 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AaEbbV8tzeCxtYUacnx0&open=AaEbbV8tzeCxtYUacnx0&pullRequest=3190

Check warning on line 339 in oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/persistentCache/AsyncNodeCache.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

"entry" is a method parameter, and should not be used for synchronization. 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AaEbbVvEzeCxtYUacnxe&open=AaEbbVvEzeCxtYUacnxe&pullRequest=3190

Check warning on line 55 in oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/persistentCache/async/CacheWriteQueueTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of thrown exception 'java.lang.Exception', as it cannot be thrown from method's body. 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AaEbbV7DzeCxtYUacnxi&open=AaEbbV7DzeCxtYUacnxi&pullRequest=3190

Check warning on line 151 in oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/persistentCache/async/CacheWriteQueueTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of thrown exception 'java.lang.Exception', as it cannot be thrown from method's body. 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AaEbbV7DzeCxtYUacnxn&open=AaEbbV7DzeCxtYUacnxn&pullRequest=3190

Check warning on line 91 in oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/persistentCache/async/CacheWriteQueueTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of thrown exception 'java.lang.Exception', as it cannot be thrown from method's body. 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AaEbbV7DzeCxtYUacnxk&open=AaEbbV7DzeCxtYUacnxk&pullRequest=3190

Check warning on line 290 in oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/DocumentCacheMaintenanceTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename "previousAsync" which hides the field declared at line 49. 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AaEbbV8tzeCxtYUacnxz&open=AaEbbV8tzeCxtYUacnxz&pullRequest=3190

Check warning on line 80 in oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/AsyncMemoryDiffCacheTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "Thread.sleep()". 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AaEbbV8_zeCxtYUacnx3&open=AaEbbV8_zeCxtYUacnx3&pullRequest=3190

Check warning on line 227 in oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/DocumentCacheMaintenanceTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename "previousAsync" which hides the field declared at line 49. 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AaEbbV8tzeCxtYUacnxy&open=AaEbbV8tzeCxtYUacnxy&pullRequest=3190

Check warning on line 386 in oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/DocumentCacheMaintenanceTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of thrown exception 'java.lang.Exception', as it cannot be thrown from method's body. 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AaEbbV8tzeCxtYUacnx2&open=AaEbbV8tzeCxtYUacnx2&pullRequest=3190

Check warning on line 60 in oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/persistentCache/CacheEntries.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this "Map.get()" and condition with a call to "Map.computeIfPresent()". 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AaEc9A_8-6uobHz6W43L&open=AaEc9A_8-6uobHz6W43L&pullRequest=3190

Check warning on line 84 in oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/persistentCache/AsyncCacheBroadcastTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "Thread.sleep()". 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AaEbbV8SzeCxtYUacnxx&open=AaEbbV8SzeCxtYUacnxx&pullRequest=3190

Check warning on line 156 in oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/persistentCache/AsyncNodeCacheTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "Thread.sleep()". 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AaEbbV8HzeCxtYUacnxv&open=AaEbbV8HzeCxtYUacnxv&pullRequest=3190

Check warning on line 402 in oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/persistentCache/AsyncNodeCacheTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this empty statement. 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AaEbbV8HzeCxtYUacnxw&open=AaEbbV8HzeCxtYUacnxw&pullRequest=3190

Check warning on line 86 in oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/persistentCache/AsyncNodeCacheTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Only one method in a class should be annotated @After. 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AaEbbV8GzeCxtYUacnxt&open=AaEbbV8GzeCxtYUacnxt&pullRequest=3190

Check warning on line 141 in oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/persistentCache/AsyncNodeCache.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this if-then-else statement by a single method invocation. 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AaEbbVvEzeCxtYUacnxd&open=AaEbbVvEzeCxtYUacnxd&pullRequest=3190

Check warning on line 170 in oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/persistentCache/AsyncMetadataQueueTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of thrown exception 'java.lang.Exception', as it cannot be thrown from method's body. 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AaEbbV7ezeCxtYUacnxo&open=AaEbbV7ezeCxtYUacnxo&pullRequest=3190

Check warning on line 570 in oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/persistentCache/AsyncMetadataQueueTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add the "@Override" annotation above this method signature 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AaEbbV7ezeCxtYUacnxr&open=AaEbbV7ezeCxtYUacnxr&pullRequest=3190

Check warning on line 72 in oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/persistentCache/AsyncNodeCacheTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused "TemporaryFolder". 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AaEbbV8HzeCxtYUacnxu&open=AaEbbV8HzeCxtYUacnxu&pullRequest=3190

Check warning on line 422 in oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/persistentCache/AsyncMetadataQueueTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "Thread.sleep()". 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AaEbbV7ezeCxtYUacnxq&open=AaEbbV7ezeCxtYUacnxq&pullRequest=3190

Check warning on line 75 in oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/persistentCache/AsyncNodeCache.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Format specifiers should be used instead of string concatenation. 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AaEbbVvEzeCxtYUacnxc&open=AaEbbVvEzeCxtYUacnxc&pullRequest=3190
View more details on SonarQubeCloud