-
Notifications
You must be signed in to change notification settings - Fork 212
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix ag-solo kernel reentrancy and fake-chain non-determinism #710
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove the test.only and make sure CI still passes afterwards.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd like to see some extra explanation per my comment, but it's not worth blocking you if that's an issue.
Propagate inboundHandler errors better. Prevent asynchronous rejection errors.
f82558d
to
c70b8a1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well OK then.
Fixes #268
This PR fixes a longstanding problem with anacrophobia in an
ag-solo
that had been subjected to HTTP calls and then restarted. It also fixes some other sources of nondeterminism in theag-solo
replay mechanism due to problems in the timer device integration.Propagate inboundHandler errors better. Prevent asynchronous rejection errors. Synchronise the inbound messages calls.