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
IGNITE-17100 Make JoinReady command idempotent #853
Conversation
|
||
assertThat(service.completeJoinCluster(), willCompleteSuccessfully()); | ||
|
||
assertThat(service.completeJoinCluster(), willCompleteSuccessfully()); |
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.
Why is completion tried 3 times, while start is only tried 2 times?
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 just wanted to verify that there's no state behind the start, so I execute the completion phase 1 time more
assertThat(service.completeJoinCluster(), willCompleteSuccessfully()); | ||
|
||
assertThat(service.completeJoinCluster(), willCompleteSuccessfully()); | ||
} |
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.
Does it make sense to also check how startJoinCluster()
works AFTER completeJoinCluster()
?
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 don't know, it shouldn't happen in a normal case scenario...
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.
LGTM
https://issues.apache.org/jira/browse/IGNITE-17100