search

DiceDB / dice

DiceDB is a redis-compliant, reactive, scalable, highly-available, unified cache optimized for modern hardware.
https://dicedb.io/
Other
6.86k stars 1.09k forks source link

Added ZADD options implementation - XX|NX|CH|INCR| LT|GT according to Redis source code #1205

Closed rushabhk04 closed 2 weeks ago

rushabhk04 commented 1 month ago

I have implemented the option parsing for the ZADD command by Redis. Additionally, I wrote unit and integration test cases to cover the functionality.

@apoorvyadav1111 Please review this and let me know if any modifications are needed. Thank you!

I added the evaluated command logic, which handles ZADD based on option flags and operates according to the Redis source code. I also created the zset_test file, which contains test cases for the command with various combinations of options.

PR for Issue https://github.com/DiceDB/dice/issues/761

Closed PR related to the above https://github.com/DiceDB/dice/pull/1138

Thanks!

JyotinderSingh commented 4 weeks ago

Thanks for this contribution, please add the respective integration tests into the resp, HTTP, and websocket directories.