As I'm on Easter holidays over here, I spent 2 hours of hacking time doing 2 things:
Adding the concept of "epochs" as you suggested, so that we can ignore stale DeadLetters. I have also cleaned up the restartChild section, got rid of the yielded Bool in favour of a more richer data structure RestartResult and finally moved emitting the events down to handleEvents, it seems just much cleaner.
I fixed a bug in handleEvent where my typeOf check was never firing. The subtleties was that what we had as input of typeOf was SomeException thus that check was always yielding False regardless which kind of Exception we had as input. Yikes!
Coverage decreased (-2.07%) to 85.621% when pulling 048c5bc24895b0e742e58553f457c0a5c575f363 on add-epoch-to-letters-and-children into da93c5f20ff36c00661ac4764a13637188cc7e66 on master.
cc @srijs
As I'm on Easter holidays over here, I spent 2 hours of hacking time doing 2 things:
DeadLetter
s. I have also cleaned up therestartChild
section, got rid of the yieldedBool
in favour of a more richer data structureRestartResult
and finally moved emitting the events down tohandleEvents
, it seems just much cleaner.handleEvent
where mytypeOf
check was never firing. The subtleties was that what we had as input oftypeOf
wasSomeException
thus that check was always yieldingFalse
regardless which kind ofException
we had as input. Yikes!