Akhileshwar Gurram
Akhileshwar Gurram
can i take this up please
Inconsistencies found only with the error messages and not in functionality. 1. Data types: Redis: "INCR strkey": "Error: value is not an integer or out of range" DiceDB: "INCR strkey":...
Having multiple PRs for this might not be a good idea. Can you take up the change in error messages as well? @vanshavenger or I can push my commits to...
@vanshavenger can you take up the changes in the error messages as well please, it's a small change in addition to the test cases you wanted to add. feel free...
I will make the necessary changes, I was waiting on vansh for an update but I'll create a PR today.
Could this be because of the error message from `sonic.Unmarshal` being different from that of `encoding/json` https://github.com/DiceDB/dice/blob/34d5fa5ae700de8d7d87e985345db0a6c00b077f/tests/json_test.go#L173-L177 the `expected` field can be set to `"ERR invalid JSON"` since we're doing...
Hello, I would like to take this up please
Is it okay if i take up #165 as well; as this one cannot be done without it; i know someone has already requested to get assigned to that issue,...
@JyotinderSingh I have made the suggested changes, also found 2 more cases where current implementation of `JSONPATH` is not supported; created test cases as you suggested. Kindly check.
@JyotinderSingh Alright, let me know if it looks good; I will keep working on #166