Add support for `HSET` Command
We need to implement the HSET command similar to the HSET command in Redis. This command sets field in the hash stored at key to value. 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.
@JyotinderSingh can you please assign this to me.
@JyotinderSingh , actually, I was reading about this command on the redis website, could you please guide that what's the use of the field option in this command? Like, HSET key field value [field value ...], in this I'm not able to understand the use of field option
@JyotinderSingh can you please assign this to me.
Would recommend taking up only one issue at a time to allow everyone a chance to contribute.
@JyotinderSingh , actually, I was reading about this command on the redis website, could you please guide that what's the use of the
fieldoption in this command? Like,HSET key field value [field value ...], in this I'm not able to understand the use offieldoption
Explore the docs and play around, that is the best way to learn.
Please assign this to me.
Please assign this to me.
Assigned
@JyotinderSingh could you please assign this to me?
@JyotinderSingh could you please assign this to me?
Assigned
Hi, @kolharsam Could you please let me know the ETA for this issue, looks like mine issue https://github.com/DiceDB/dice/issues/257 is dependent on this issue implementation. cc @JyotinderSingh
Hi, @kolharsam Could you please let me know the ETA for this issue, looks like mine issue https://github.com/DiceDB/dice/issues/257 is dependent on this issue implementation. cc @JyotinderSingh
Hi @raghavbabbar sorry about the delay. Had a very busy weekend. Will update regarding this in a couple of days
@kolharsam Are you actively working on this?
Yes, @JyotinderSingh I am actively working on this. Will share some proper updates soon.
Yes, @JyotinderSingh I am actively working on this. Will share some proper updates soon.
Awesome, thanks!