Skip to content
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

Move mpool clear to lotus-shed or hide it #4854

Closed
jennijuju opened this issue Nov 13, 2020 · 3 comments · Fixed by #5900
Closed

Move mpool clear to lotus-shed or hide it #4854

jennijuju opened this issue Nov 13, 2020 · 3 comments · Fixed by #5900
Assignees
Labels
area/ux Area: UX

Comments

@jennijuju
Copy link
Member

Many people uses lotus mpool clear wrongly/without knowing what they should do with it. Normally they will use it to clear the mpool when the mpool is stuck and then get message issues.

We should limit hide this cmd so that people dont use it randomly.

@jennijuju jennijuju added the area/ux Area: UX label Nov 13, 2020
@f8-ptrk
Copy link
Contributor

f8-ptrk commented Nov 16, 2020

you should not hide a command that is available. you should clearly explain what it's doing and what the consequences are and let people decide if they want to use it or not.

@arajasek
Copy link
Contributor

Moving this to lotus-shed is probably the right thing to do.

@placer14 placer14 self-assigned this Mar 29, 2021
@arajasek
Copy link
Contributor

Make hidden / mark as deprecated initially when it moves to lotus-shed, delete later?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ux Area: UX
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants