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
tikv-ctl: add bad-regions
subcommand.
#2624
Conversation
can you give me some examples, like damaging some region metas and using our tool? |
I have add some test cases to show how the command check bad regions. PTAL thanks. |
…v-ctl-bad-regions
src/bin/tikv-ctl.rs
Outdated
@@ -490,6 +492,10 @@ impl DebugExecutor for DebugClient { | |||
fn set_region_tombstone(&self, _: u64, _: Region) { | |||
unimplemented!(); | |||
} | |||
|
|||
fn print_bad_regions(&self) { | |||
panic!("only avaliable for local mode"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about unimplemented!
?
Great work |
@overvenus PTAL again, thanks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
/run-all-tests |
close #2612
Now the command can check all regions which have corrupted
last index
,commited index
orapplied index
. But it can't handle bad snapshot until now.