This is an automated email from the ASF dual-hosted git repository. daim pushed a commit to branch DetailedGC/OAK-10199 in repository https://gitbox.apache.org/repos/asf/jackrabbit-oak.git
commit bfc42513659155bf53f3e61580af02305791bd23 Author: Rishabh Kumar <[email protected]> AuthorDate: Tue Oct 3 19:46:58 2023 +0530 OAK-8646 : fixed sonar issue of operating of ints to avoid overflow --- .../jackrabbit/oak/plugins/document/VersionGarbageCollector.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/VersionGarbageCollector.java b/oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/VersionGarbageCollector.java index 9b4cdff6b4..cd0de7c88e 100644 --- a/oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/VersionGarbageCollector.java +++ b/oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/VersionGarbageCollector.java @@ -1199,7 +1199,8 @@ public class VersionGarbageCollector { detailedGCStats.propertiesDeleted(deletedProps); detailedGCStats.unmergedBranchCommitsDeleted(deletedUnmergedBCSet.size()); detailedGCStats.documentsUpdated(updatedDocs); - detailedGCStats.documentsUpdateSkipped(oldDocs.size() - updatedDocs); + // fix for sonar : converted to long before operation + detailedGCStats.documentsUpdateSkipped((long)oldDocs.size() - updatedDocs); } finally { // now reset delete metadata updateOpList.clear();
