We may need to build a Redis command compatibility test for Kvrocks
- Dominant language
- C++
- Stars
- 4.4k
- Forks
- 658
- Avg merge
- 1d 20h
- Merged PRs (30d)
- 10
Description
### Search before asking
- [x] I had searched in the [issues](https://github.com/apache/kvrocks/issues) and found no similar issues.
### Motivation
From a design perspective, Kvrocks should maintain full backward compatibility with Redis. For commands that exist in Redis, Kvrocks needs to have consistent external behavior
However, during testing, I discovered that the TDIGEST.MERGE command has a compatibility issue, refer to: https://github.com/apache/kvrocks/issues/3410
Even the smartest and most technically capable people make mistakes.
Therefore, we should integrate compatibility testing into CI to verify the behavior of Redis and Kvrocks for the same Redis commands, and continuously enrich the test suite.
### Solution
Establish compatibility testing in CI and continuously enrich the test suite.
### Are you willing to submit a PR?
- [x] I'm willing to submit a PR!
Contributor guide
No contributing guide indexed for this repository
Research direction
Start by reading issue 3410 and comparing the TDIGEST.MERGE behavior in Redis and Kvrocks. Then locate the existing command tests and CI entry points, and define the compatibility coverage and CI checks needed for the work to be complete.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp, redis
- Domain
- ci-cd, databases, testing
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Quiet
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100