apache / apache/kvrocks

TimeSeries: Implement `TS.DELETERULE` command

Open
#3,210 4 comments 0 reactions 1 assignee Claimed by @var-nan View on GitHub
good first issue help wanted
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

We should implement `TS.DELETERULE` command.

Redis command reference:
https://redis.io/docs/latest/commands/ts.deleterule/

### Solution

Refer to `TS.CREATERULE`(https://github.com/apache/kvrocks/pull/3151/commits/9da1c2c8407edccc539b0e52b3a508a856f34462 , https://github.com/apache/kvrocks/pull/3151/commits/264a122b2aa04b4fc7afec4e9039c0ccc22222a8)

- Delete the related `DOWNSTREAM` subkey, modify the `sourceKey` field in metadata
- Add `TS.DELETERULE` command
- Add go integrated test

### Are you willing to submit a PR?

- [ ] I'm willing to submit a PR!

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.