Branch: refs/heads/CRAPP-438 Home: https://github.com/xwiki-contrib/application-changerequest Commit: 9e88c569a18b41f9a73e0259a78a83c754304a9f https://github.com/xwiki-contrib/application-changerequest/commit/9e88c569a1... Author: Simon Urli <[email protected]> Date: 2026-09-18 (Fri, 18 Sep 2026) Changed paths: M application-changerequest-default/src/main/java/org/xwiki/contrib/changerequest/internal/DefaultChangeRequestManager.java M application-changerequest-default/src/test/java/org/xwiki/contrib/changerequest/internal/DefaultChangeRequestManagerTest.java Log Message: ----------- CRAPP-438: Review invalidation is skipped when a stale change request is served from the storage cache * Remove the warning logged in DefaultChangeRequestManager#addReview when the change request held fewer reviews than the storage contains: the reload performed just before repairs that state, so the warning reported a condition that the same method had already corrected * What remains detectable that way is a change request loaded before a concurrent save, which is ordinary staleness under concurrency rather than a fault, while an outdated entry holding the right number of reviews with the wrong validity went unnoticed anyway, so the comparison was both noisy and blind * Drop the log assertions from the addReview unit test accordingly, which keeps asserting the invalidation of the previous review, both saves and the resulting review list, and still fails when the reload is removed Co-Authored-By: Claude Opus 5 (1M context) <[email protected]> To unsubscribe from these emails, change your notification settings at https://github.com/xwiki-contrib/application-changerequest/settings/notifica...