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.

Sign up for GitHub

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

Jump to bottom

dss: fix kvraft tests #427

Closed

makisevon wants to merge 1 commit into pingcap:master from makisevon:fix

makisevon commented Jun 15, 2022

What problem does this PR solve?

Issue number: close DSS: There is something strange about the tests in kvraft (lab3). #426
Breif description of the problem: Remove some illogical limitations.

What is changed and how it works?

Replace cfg.net.spawn with thread::spawn to avoid EnterError.
Not to use Arc::try_unwrap since it may return Err.

Check List

Tests

Unit test


          fix(dss): kvraft tests

c09923f

This pull request was closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment