chesnokoff commented on code in PR #13136:
URL: https://github.com/apache/ignite/pull/13136#discussion_r3309174423
##########
modules/core/src/test/java/org/apache/ignite/internal/processors/cache/transactions/TxDeadlockDetectionNoHangsTest.java:
##########
@@ -68,6 +74,9 @@ public class TxDeadlockDetectionNoHangsTest extends
GridCommonAbstractTest {
cfg.setCacheConfiguration(ccfg);
+ listeningLog.registerListener(lsnr);
Review Comment:
I mean that `lsnr.reset()` can clear the listener state. Can
`getConfiguration()` be called after `Deadlock detection was timed out` was
already logged?
`doTest()` runs for about 120 seconds. During this time, the restart thread
starts `NODES_CNT` grids many times, and each start calls `getConfiguration()`.
The deadlock detection timeout is 60 seconds, so the timeout log can appear
first. After that, another node restart can call `getConfiguration()` and reset
the listener before `afterTest()` calls `lsnr.check()`
--
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]
