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
[Merged by Bors] - Avoid races in postVerifier #4965
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #4965 +/- ##
=========================================
- Coverage 77.2% 77.2% -0.1%
=========================================
Files 254 254
Lines 30081 30092 +11
=========================================
+ Hits 23232 23235 +3
- Misses 5349 5357 +8
Partials 1500 1500
|
4baf2d2
to
096916b
Compare
8ceaa93
to
5a22887
Compare
bors merge |
## Motivation Closes #4910, merge after spacemeshos/post#232. ## Changes - Removed `Start` method, a verifier returned from `NewOffloadingPostVerifier` is ready to be used - `Close` cancels ongoing verifications and avoids new ones being started - Updated `node.go` to use the new API ## Test Plan - existing tests pass ## TODO <!-- This section should be removed when all items are complete --> - [x] Explain motivation or link existing issue(s) - [x] Test changes and document test plan - [x] Update documentation as needed - [x] Update [changelog](../CHANGELOG.md) as needed
5a22887
to
cc8c7fc
Compare
Canceled. |
bors merge |
## Motivation Closes #4910, merge after spacemeshos/post#232. ## Changes - Removed `Start` method, a verifier returned from `NewOffloadingPostVerifier` is ready to be used - `Close` cancels ongoing verifications and avoids new ones being started - Updated `node.go` to use the new API ## Test Plan - existing tests pass ## TODO <!-- This section should be removed when all items are complete --> - [x] Explain motivation or link existing issue(s) - [x] Test changes and document test plan - [x] Update documentation as needed - [x] Update [changelog](../CHANGELOG.md) as needed
Pull request successfully merged into develop. Build succeeded! The publicly hosted instance of bors-ng is deprecated and will go away soon. If you want to self-host your own instance, instructions are here. If you want to switch to GitHub's built-in merge queue, visit their help page. |
Motivation
Closes #4910, merge after spacemeshos/post#232.
Changes
Start
method, a verifier returned fromNewOffloadingPostVerifier
is ready to be usedClose
cancels ongoing verifications and avoids new ones being startednode.go
to use the new APITest Plan
TODO