Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

revert command doesn't support --database paritydb #14046

Closed
2 tasks done
crystalin opened this issue Apr 29, 2023 · 0 comments · Fixed by #14182
Closed
2 tasks done

revert command doesn't support --database paritydb #14046

crystalin opened this issue Apr 29, 2023 · 0 comments · Fixed by #14182
Labels
J2-unconfirmed Issue might be valid, but it’s not yet known.

Comments

@crystalin
Copy link
Contributor

Is there an existing issue?

  • I have searched the existing issues

Experiencing problems? Have you tried our Stack Exchange first?

  • This is not a support question.

Description of bug

The revert command, which allows to removing some of the last imported blocks, doesn't support providing --database and will default to rocksdb, making it impossible to revert when using paritydb.

Steps to reproduce

>revert 100  --base-path /home/networks --blocks-pruning archive --state-pruning archive  -l trace --database paritydb
error: unexpected argument '--database' found
@juangirini juangirini added the J2-unconfirmed Issue might be valid, but it’s not yet known. label May 17, 2023
@bkchr bkchr mentioned this issue May 20, 2023
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
J2-unconfirmed Issue might be valid, but it’s not yet known.
Projects
SDK Node
Status: done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

RevertCmd: Expose database params via CLI paritytech/substrate
2 participants
@crystalin @juangirini