-
Notifications
You must be signed in to change notification settings - Fork 492
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
Extract round related actions to method #9705
Conversation
@kiminuo @adamPetho can you take a quick look at this? |
@@ -234,16 +232,50 @@ public async Task<CoinJoinResult> StartRoundAsync(IEnumerable<SmartCoin> smartCo | |||
_ => throw new ArgumentOutOfRangeException() | |||
}; | |||
roundState.LogInfo(msg); |
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.
So if ProceedWithRoundAsync
throws an exception that is caught by catch (UnexpectedRoundPhaseException ex) when (ex.Actual == Phase.Ended)
, then here we can still log that the round finished with broadcasting a transaction, right?
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.
Whatever the coordinator indicates with EndRoundState
.
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.
tACK on testnet, but please resolve Kimi's suggestions.
Co-authored-by: Kimi <58662979+kiminuo@users.noreply.github.com>
…into extractround
LGTM now |
Tested this during the night. It works reliably. |
Preparation for this #9681
Just moving the code and a small restructure.