-
Notifications
You must be signed in to change notification settings - Fork 49
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
Feature/reflection #76
Conversation
// Abort resets the upload schedule and cancels blob upload | ||
// after the currently uploading blob is finished. | ||
func (r *Manager) Abort() { | ||
r.abortTimer <- true |
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.
have you looked at https://github.com/lbryio/lbry.go/tree/master/extras/stop ?
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 have, but cannot figure out how it would be beneficial to use here instead of two plain channels. Might be a different story if uploads need to be parallelized later.
Changed to use public API. I decided that to make any optimizations like parallel uploads at this point would be premature.