-
Notifications
You must be signed in to change notification settings - Fork 468
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
Add basic tests for kvrocks2redis #1338
Comments
It should be hard to add Go test cases for this tool but can add a fixed dataset to make sure it works well with those data types. |
@git-hulk That can be the case. But before we dive into the implementation, I suggest we reach a consensus on what fundamental functions of kvrocks2redis are and how a user manually use it. |
OK, let me add a how-to-use kvrocks2redis doc first. |
@git-hulk do you have a plan or estimate on this ticket? |
@tisonkun Yes, I wonder if I should refactor this tool since it's NOT easy to maintain now. But I haven't started it yet. Plan to do that at the beginning of May, after I released the first version of the controller: apache/kvrocks-controller#49 |
@git-hulk can you provide some basic how-to-use info here so that I can follow up? |
Sure, thanks. Sorry for forgetting this, I'll add the how-to-use doc this week first. |
Related to apache/kvrocks-website#138. |
I also created another issue #1643 to track improvements of |
Search before asking
Motivation
So that we're sure that the fundamental functions of kvrocks2redis work well.
Solution
@git-hulk I'm still counting on you to understand how to use kvrocks2redis. So that we can add some gocase to cover those usages.
Are you willing to submit a PR?
The text was updated successfully, but these errors were encountered: