-
Notifications
You must be signed in to change notification settings - Fork 82
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: Race conditions #178
Fix: Race conditions #178
Conversation
…rted Member methode on Node
…ventEmitter in applicable structs
@@ -28,9 +28,6 @@ env: | |||
- RUN="make test-examples" | |||
|
|||
matrix: | |||
allow_failures: |
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.
✨ 💖 ✨
Looks really good overall. Some small questions and nits. |
a2b640f
to
ded6b51
Compare
…e to prevent race conditions
909bf5e
to
4290475
Compare
…ingpop for the time being to help easy upgrading.
4290475
to
cc23f0c
Compare
@@ -24,6 +24,8 @@ import ( | |||
"sort" | |||
"testing" | |||
|
|||
"github.com/benbjohnson/clock" | |||
|
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.
Remove white line
return stop, wait | ||
} | ||
|
||
// TODO add schedule once with a quit channel to cancel it for use in the update rollup |
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.
TODO
There are some todo's left, but besides that: LGTM! |
@@ -27,9 +27,6 @@ env: | |||
- RUN="make test-race" | |||
|
|||
matrix: | |||
allow_failures: |
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 |
This PR is to remove the races from ringpop-go and let travis actually fail when there are race conditions in ringpop.