-
Notifications
You must be signed in to change notification settings - Fork 199
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
Implement go routines closing mechanism #1742
Conversation
SebastianMarian
commented
May 13, 2020
•
edited
Loading
edited
- Added go routines close mechanism through context and io.Close implementation for components: Chronology, Worker, SyncTimer, Bootstrapper, epochStartTrigger, miniBlocksPoolsCleaner, txsPoolsCleaner
…ementation for components: Chronology, Worker, SyncTimer, Bootstrapper
consensus/spos/interface.go
Outdated
@@ -96,6 +98,7 @@ type SubroundsFactory interface { | |||
|
|||
//WorkerHandler represents the interface for the SposWorker | |||
type WorkerHandler interface { | |||
io.Closer |
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.
shouldn't we use our own interface ? instead of io.Closer
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.
yep, removed it from all modified interfaces
…entation for components: epochStartTrigger, txsPoolsCleaner, miniBlocksPoolsCleaner
…d checkChannels from Worker
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.
System tests passed.