apache / apache/kvrocks

We may need to build a Redis command compatibility test for Kvrocks

Open
#3,411 4 comments 0 reactions 0 assignees View on GitHub
enhancement
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.