Add support for `PX` in `SET` command
We already have the SET command implemented. Now we need to add support for PX option similar to the PX option in Redis. Please refer to the following commit in Redis to understand the implementation specifics - source.
Write unit and integration tests for the command referring to the tests written in the Redis codebase 7.2.5. For integration tests, you can refer to the tests folder. Note: they have used TCL for the test suite, and we need to port that to our way of writing integration tests using the relevant helper methods. Please refer to our tests directory.
For the command, benchmark the code and measure the time taken and memory allocs using benchmem and try to keep them to the bare minimum.
Hey @arpitbbhayani , I would like to take this up.
@SyedMa3 Assigned the issue to you. Go for it.
It would be awesome if you keep posting updates to this issue so that we know you are working on it. If we see an inactivity of 7 days, we will remove your assignment. This is just to ensure that the project maintains its momentum and others are not blocked on this work.
But it really means a ton to us that you are contributing to this project. Thanks again.
Sure, understandable.
completed the task. Waiting for Issue 213 to get resolved.
Side Note: The benchmark cli command for
evalSETneeds to have the value ofbenchtimeflag less than100xsinceWatchChannelis a buffered channel of size 100 and may lead to a deadlock.
https://github.com/DiceDB/dice/pull/236/