-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Replace keydb with kvrocks #7339
Conversation
7944ad9
to
4d62105
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## develop #7339 +/- ##
===========================================
- Coverage 83.26% 83.25% -0.01%
===========================================
Files 373 373
Lines 39615 39615
Branches 3701 3701
===========================================
- Hits 32984 32981 -3
- Misses 6631 6634 +3
|
Please add a changelog entry. |
Glad to see CVAT choose Apache Kvrocks as the on-disk Redis solution. Would you mind to add you in Kvrocks Users page? I prepare a PR for you to preview apache/kvrocks-website#200 but your opinion is significant and perhaps you'd propose a different brand kit. |
@tisonkun , thank you! Please download icons from here https://www.cvat.ai/media. We would prefer to use the icon below: ![]() |
Hi @nmanovic! @bsekachev suggested a different logo at apache/kvrocks-website#200 (comment). Please take a look if there is unaligned. |
Motivation and context
Resolved #7345
How has this been tested?
Checklist
develop
branch(cvat-canvas,
cvat-core,
cvat-data and
cvat-ui)
License
Feel free to contact the maintainers if that's a concern.