He-Pin opened a new pull request, #3436: URL: https://github.com/apache/pekko/pull/3436
### Motivation The "should not interfere with receive timeouts" test previously only verified that the receive timeout value is preserved after recovery. It did not verify that a short receive timeout doesn't accidentally trigger the recovery timeout, nor that the recovery timeout still fires when the journal is unresponsive. ### Modification - Use separate probes for persist and replay phases to avoid cross-contamination. - Use a short receive timeout (20ms) during replay to verify it does not trigger recovery timeout (`expectNoMessage`). - Verify that recovery timeout (500ms) still fires when the journal does not respond (`expectMsgType[Failure]` with `RecoveryTimedOut`). - Reduce `recovery-event-timeout` from 30s to 500ms for the receive-timeout journal to keep the test fast. - Remove unused `TestDuration` import. ### Result The test now encodes the intended contract: receive timeout and recovery timeout are independent mechanisms. The test is deterministic and fast (~700ms total for the replay phase). ### Tests - `sbt "persistence / Test / testOnly org.apache.pekko.persistence.PersistentActorRecoveryTimeoutSpec"` ### References Refs akka/akka-core#32030 -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
