DiceDB
Add support for command `BITFIELD_RO`
Add support for the BITFIELD_RO command in DiceDB similar to the BITFIELD_RO command 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.
@arpitbbhayani i would like to work on this, can you please assign to me?
hi @kushal0511-not, are you still working on this issue? If not, Can I pick this up?
@anchalsinghrajput i am waiting for #639 to complete , this command will use that command only.
Hi @kushal0511-not , #639 is in progress and will need some time to ensure I handle all the edge cases for all 4 subcommands. Thanks.
Hi @kushal0511-not , I have created a PR #723 for #639. However, It is still a draft so more changes might come upon further testing. Apoorv
@JyotinderSingh I’m currently occupied with other work commitments and won’t be able to work on this. Feel free to assign someone else. Thanks.
Hey @JyotinderSingh , will get this done since Kushal has unassigned himself. Please assign to me!
Hey @JyotinderSingh , will get this done since Kushal has unassigned himself. Please assign to me!
Assigned
Just an update. Have implemented the methods, and added tests. Will be raising PR, post adding some more tests.
Hi @iamskp11, bitfield has been implemented. This issue was tagged as dependent on the bitfield command, so just wanted to share update here.