Append command is throwing error with ZADD

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

Append command is throwing error with ZADD #1233

Open shashi-sah2003 opened 3 weeks ago

shashi-sah2003 commented 3 weeks ago

This is Draft PR for issue #1193

c-harish commented 3 weeks ago

Hi, @shashi-sah2003. APPEND implementation is checking only encoding. Please check for the type as well and make it consistent with redis.

JyotinderSingh commented 2 weeks ago

Hi @shashi-sah2003, hope you are doing well. Hope you are not blocked by anything on this PR.

shashi-sah2003 commented 2 weeks ago

Hi @shashi-sah2003, hope you are doing well. Hope you are not blocked by anything on this PR.

hey @JyotinderSingh https://github.com/DiceDB/dice/pull/1237 this is the PR I have raised, pls review it and let me know if there is anything more I need to do. thanks