Branch: refs/heads/XCOMMONS-2092 Home: https://github.com/xwiki/xwiki-commons Commit: ce8c49fb9ea2903bb76c988296fce7937877ab33 https://github.com/xwiki/xwiki-commons/commit/ce8c49fb9ea2903bb76c988296fce7... Author: Simon Urli <[email protected]> Date: 2026-09-17 (Thu, 17 Sep 2026) Changed paths: M xwiki-commons-core/xwiki-commons-job/xwiki-commons-job-default/src/main/java/org/xwiki/job/internal/ReadWriteSemaphore.java M xwiki-commons-core/xwiki-commons-job/xwiki-commons-job-default/src/test/java/org/xwiki/job/internal/DefaultJobExecutorTest.java Log Message: ----------- XCOMMONS-2092: Fix new SonarCloud findings from the previous commit * java:S2273 on ReadWriteSemaphore#waitUninterruptibly(): both call sites already hold the monitor's lock before calling this private method, but Sonar cannot verify that across the method boundary. Suppressed with a comment explaining why. * java:S2925 on the Thread.sleep() in matchingGroupPathAreBlockedPoolMultiSizeParentFirst: there is no observable state change to poll on instead, as already explained by the comment above the call (and as already accepted, unsuppressed, for the identical pattern in matchingGroupPathAreBlockedPoolMultiSizeChildrenFirst). Suppressed at the test method. * java:S1134 on the same test: dropped the redundant "FIXME:" comment line, since the paragraph above it already explains the constraint in full - it was never an outstanding task, just copied wording from the pre-existing pattern in the sibling test method. Co-Authored-By: Claude Sonnet 5 <[email protected]> To unsubscribe from these emails, change your notification settings at https://github.com/xwiki/xwiki-commons/settings/notifications