It will change the interface of failpoint.Disable but not failpoints.Disable and thus almost usage should be fine.
failpoint.Disable now never return an error because it always succeeds now. If we want to be backward compatible on compilation (but we don't even do a semver release), we can keep the interface and return nil always.
tisonkun
commented
2 years ago
This closes #75.
It will change the interface of
failpoint.Disablebut notfailpoints.Disableand thus almost usage should be fine.failpoint.Disablenow never return an error because it always succeeds now. If we want to be backward compatible on compilation (but we don't even do a semver release), we can keep the interface andreturn nilalways.Signed-off-by: tison wander4096@gmail.com