Support failpoints group

tikv / fail-rs

Fail points for rust

Apache License 2.0

332 stars 39 forks source link

Support failpoints group #55

Open hunterlxt opened 4 years ago

hunterlxt commented 4 years ago

Let multiple tests run over different threads, meanwhile each configuration doesn't affect others.

expose FailPointRegistry as local registry.
pub fn create_registry() -> FailPointRegistry can get a new FailPointRegistry. So we can bound threads into exact one registry with register_current(&self) and deregister_current(&self).
add a test to show multiple threads don't affect each other.

close #51

hunterlxt commented 4 years ago

I think all the tests need to be updated, It is better to test in the form of fail_points!. But I'm busy for special week now, and will continue to improve unit testing later.

hunterlxt commented 4 years ago

@BusyJay I think this PR is ready PTAL

zhangjinpeng87 commented 4 years ago

@NingLin-P PTAL

hunterlxt commented 4 years ago

This PR is staled for about a week

BusyJay commented 4 years ago

So have you tried to use it in TiKV? Is everything working well?

hunterlxt commented 4 years ago

https://github.com/tikv/tikv/pull/8065 @BusyJay