-
Notifications
You must be signed in to change notification settings - Fork 12
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
Improve fwdctl dump
#348
Comments
import from json (or any format) would also be amazing |
@rkuris are there any unit or integration tests that cover the relevant flow here to help get my head around the stack? Or perhaps the examples/insert.rs might be relevant to look at? Thanks. |
@rkuris Hi Ron, I am trying to work on this change and have a question regarding to "if we stopped, have an option to render the next key for resuming".
Thanks! |
Let's say you have keys A to Z and ask for a limit of 10 keys. In that
case, it would be nice if it told you that you can get the next set by
starting at K (the 11th key). I imagine the output could look like this:
A
B
...
J
Next key is K, resume with "--start-key=K"
…On Mon, Dec 16, 2024, 9:28 PM zdf ***@***.***> wrote:
@rkuris <https://github.com/rkuris> Hi Ron, I am trying to work on this
change and have a question regarding to "if we stopped, have an option to
render the next key for resuming".
1. Does the stopping here mean interrupt in the middle / crush or it
reached the stopped key or max number of keys?
2. Is storing a temporary file locally a good approach to resume? or
do you have any ideas in mind?
Thanks!
—
Reply to this email directly, view it on GitHub
<#348 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAYLR3FBOLLEWZQOLRSKT3D2F6Y7RAVCNFSM6AAAAABSGKBPZ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNBXGUZTQMRYGI>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Got it. Thanks for the explanation! |
Right now, the dump command only takes a database name. It would be easy to add options now:
Some of these stream limits are already implemented for range proofs, so you can use the code there as a template of how to stop iterating.
(depends on #347)
The text was updated successfully, but these errors were encountered: