Skip to content

OAK-8840: copy jackrabbit-data into oak-blob #2810

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
reschke wants to merge 15 commits into
base: trunk
Choose a base branch
from

OAK-8840: copy jackrabbit-data into Oak: rename *this* LazyFileInpuSt…

1b2fc44
Select commit
Loading
Failed to load commit list.
Open

OAK-8840: copy jackrabbit-data into oak-blob #2810

OAK-8840: copy jackrabbit-data into Oak: rename *this* LazyFileInpuSt…
1b2fc44
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Mar 25, 2026 in 5m 44s

Quality Gate failed

Failed conditions
1 Security Hotspot
43.0% Coverage on New Code (required ≥ 80%)
E 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 230 in oak-blob/src/test/java/org/apache/jackrabbit/oak/spi/blob/data/TestCaseBase.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=AZ0b1EOAOyJaQ2D5roZX&open=AZ0b1EOAOyJaQ2D5roZX&pullRequest=2810

Check warning on line 1359 in oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/data/CachingDataStore.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Disambiguate this call by either casting as "Object" or "Object[]". 

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

Check warning on line 81 in oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/data/AbstractDataRecord.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this instanceof check and cast with 'instanceof DataRecord datarecord' 

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

Check warning on line 401 in oak-blob/src/test/java/org/apache/jackrabbit/oak/spi/blob/data/TestCaseBase.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=AZ0b1EOAOyJaQ2D5roZe&open=AZ0b1EOAOyJaQ2D5roZe&pullRequest=2810

Check warning on line 156 in oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/data/MultiDataStore.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Make this final field static too. 

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

Check warning on line 171 in oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/fs/local/HandleMonitor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Invoke method(s) only conditionally. Use the built-in formatting to construct this argument. 

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

Check warning on line 164 in oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/fs/local/LocalFileSystem.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=AZ0b1EKvOyJaQ2D5roVY&open=AZ0b1EKvOyJaQ2D5roVY&pullRequest=2810

Check warning on line 458 in oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/data/LocalCache.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the boxing to "Double". 

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

Check warning on line 450 in oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/data/FSBackend.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use "java.nio.file.Files#delete" here for better messages on error conditions. 

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

Check warning on line 318 in oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/data/MultiDataStore.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename this variable to not match a restricted identifier. 

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

Check warning on line 623 in oak-blob/src/test/java/org/apache/jackrabbit/oak/spi/blob/data/TestCaseBase.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this block of code, fill it in, or add a comment explaining why it is empty. 

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

Check failure on line 307 in oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/data/AsyncUploadCache.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use try-with-resources or close this "BufferedInputStream" in a "finally" clause. 

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

Check failure on line 176 in oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/data/FileDataStore.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Change this "try" to a try-with-resources. 

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

Check warning on line 358 in oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/data/MultiDataStore.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=AZ0b1ELTOyJaQ2D5roVw&open=AZ0b1ELTOyJaQ2D5roVw&pullRequest=2810

Check warning on line 124 in oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/data/AutoClosingLazyFileInputStream.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=AZ0mi6WbWQuldiv-5q9s&open=AZ0mi6WbWQuldiv-5q9s&pullRequest=2810

Check warning on line 190 in oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/fs/FileSystemPathUtil.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use "isEmpty()" to check whether a "String" is empty or not. 

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

Check warning on line 242 in oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/data/AsyncUploadCache.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this hard-coded path-delimiter. 

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

Check warning on line 380 in oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/data/FileDataStore.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace the type specification in this constructor call with the diamond operator ("<>"). 

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

Check warning on line 127 in oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/fs/local/HandleMonitor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace the type specification in this constructor call with the diamond operator ("<>"). 

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

Check warning on line 588 in oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/data/MultiDataStore.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Extract this nested try block into a separate method. 

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

Check warning on line 248 in oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/data/AsyncUploadCache.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace the type specification in this constructor call with the diamond operator ("<>"). 

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

Check warning on line 38 in oak-blob/src/test/java/org/apache/jackrabbit/oak/spi/blob/data/TestCachingFDS.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=AZ0b1EOxOyJaQ2D5roZ7&open=AZ0b1EOxOyJaQ2D5roZ7&pullRequest=2810

Check warning on line 623 in oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/data/LocalCache.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=AZ0b1ELrOyJaQ2D5roWr&open=AZ0b1ELrOyJaQ2D5roWr&pullRequest=2810

Check warning on line 346 in oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/data/CachingDataStore.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use isEmpty() to check whether the collection is empty or not. 

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

Check warning on line 337 in oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/data/FileDataStore.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=AZ0b1EL3OyJaQ2D5roW6&open=AZ0b1EL3OyJaQ2D5roW6&pullRequest=2810
View more details on SonarQubeCloud