github-actions[bot] commented on PR #3005:
URL: https://github.com/apache/jackrabbit-oak/pull/3005#issuecomment-4894780886
# Commit-Check ❌
```
Commit rejected by Commit-Check.
(c).-.(c) (c).-.(c) (c).-.(c) (c).-.(c) (c).-.(c)
/ ._. \ / ._. \ / ._. \ / ._. \ / ._. \
__\( C )/__ __\( H )/__ __\( E )/__ __\( C )/__ __\( K )/__
(_.-/'-'\-._)(_.-/'-'\-._)(_.-/'-'\-._)(_.-/'-'\-._)(_.-/'-'\-._)
|| E || || R || || R || || O || || R ||
_.' '-' '._ _.' '-' '._ _.' '-' '._ _.' '-' '._ _.' '-' '._
(.-./`-´\.-.)(.-./`-´\.-.)(.-./`-´\.-.)(.-./`-´\.-.)(.-./`-´\.-.)
`-´ `-´ `-´ `-´ `-´ `-´ `-´ `-´ `-´ `-´
Commit rejected.
Type message check failed ==> OAK-XXXXX: Remove ServiceListener after
awaitServiceEvent completes
The EventServiceListener was never removed from the BundleContext after
awaitServiceEvent completed (either successfully or via timeout). This
caused:
1. Resource leaks - listeners accumulated across test runs
2. Test interference - listeners from previous tests continued receiving
events
3. Intermittent test failures - unrelated service events could be captured,
causing timeout errors when waiting for specific service events
Fix: Add try-finally block to ensure bundleContext.removeServiceListener()
is always called after the await completes, preventing listener accumulation
and cross-test event interference.
Co-authored-by: Copilot <[email protected]>
The commit message should follow Conventional Commits. See
https://www.conventionalcommits.org
Suggest: Commit message does not match the required pattern
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
