Closed fuweid closed 5 years ago
Signed-off-by: Wei Fu fuweid89@gmail.com
use new context to do the cleanup
none
no need
:exclamation: No coverage uploaded for pull request head (me-dont-use-cancel-context@5b0d2ed). Click here to learn what that means. The diff coverage is n/a.
me-dont-use-cancel-context@5b0d2ed
n/a
please rebase to master
Signed-off-by: Wei Fu fuweid89@gmail.com
Ⅰ. Describe what this PR did
use new context to do the cleanup
Ⅱ. Does this pull request fix one issue?
none
Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗?)
no need
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews