-
-
Notifications
You must be signed in to change notification settings - Fork 35
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
Implement randomkey command #202
Conversation
…y, in case it gets deleted, and to ensure that the copied data are the original key from the hashtable
…e get key operation
Codecov ReportBase: 80.84% // Head: 80.87% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #202 +/- ##
==========================================
+ Coverage 80.84% 80.87% +0.04%
==========================================
Files 129 131 +2
Lines 7760 7801 +41
==========================================
+ Hits 6273 6309 +36
- Misses 1487 1492 +5
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
This PR implements the RANDOMKEY command.
It refactors the hashtable op get key function to return a copy of the key from the hashtable and to ensure that the copy is actually valid implementing a 2-phase verification.
The PR also includes the tests for the new command and a minor improvement to the DBSIZE command