Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

rawkv: ttl-check-poll-interval does not take effect on RawKV API v2 #15142

Closed
pingyu opened this issue Jul 17, 2023 · 0 comments · Fixed by #15143
Closed

rawkv: ttl-check-poll-interval does not take effect on RawKV API v2 #15142

pingyu opened this issue Jul 17, 2023 · 0 comments · Fixed by #15143
Assignees
@pingyu
Labels
affects-6.5 affects-7.1 severity/major type/bug Type: Issue - Confirmed a bug

Comments

@pingyu
Copy link
Contributor

pingyu commented Jul 17, 2023 •
edited

Bug Report

What version of TiKV are you using?

master (c27b430)

What operating system and CPU are you using?

Steps to reproduce

  1. Set storage.ttl-check-poll-interval to 24h.
  2. Put RawKV data with TTL of 7 days and incremental keys. Data size is generally similar everyday.

What did you expect?

Size of storage is approximately equal for every 7 days.

What did happened?

Size of storage keeps increasing.
@pingyu pingyu added the type/bug Type: Issue - Confirmed a bug label Jul 17, 2023
@pingyu pingyu self-assigned this Jul 17, 2023
@pingyu pingyu mentioned this issue Jul 17, 2023
Merged
ti-chi-bot bot added a commit that referenced this issue Jul 18, 2023
@pingyu @ti-chi-bot
rawkv: fix ttl_checker for RawKV API v2 (#15143)
4a7b248 
close #15142

rawkv: fix ttl_checker for RawKV API v2.

Signed-off-by: Ping Yu <yuping@pingcap.com>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
@ti-chi-bot ti-chi-bot mentioned this issue Jul 18, 2023
Merged
ti-chi-bot pushed a commit to ti-chi-bot/tikv that referenced this issue Jul 18, 2023
@pingyu @ti-chi-bot
This is an automated cherry-pick of tikv#15143
8360d95 
close tikv#15142

Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
@ti-chi-bot ti-chi-bot mentioned this issue Jul 18, 2023
Merged
ti-chi-bot bot pushed a commit that referenced this issue Jul 19, 2023
@ti-chi-bot @pingyu
rawkv: fix ttl_checker for RawKV API v2 (#15143) (#15154)
a8bd87c 
close #15142

rawkv: fix ttl_checker for RawKV API v2.

Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
Signed-off-by: Ping Yu <yuping@pingcap.com>

Co-authored-by: Ping Yu <yuping@pingcap.com>
pingyu added a commit to pingyu/tikv that referenced this issue Jul 19, 2023
@ti-chi-bot @pingyu
rawkv: fix ttl_checker for RawKV API v2 (tikv#15143) (tikv#15154)
61d1d98 
close tikv#15142

rawkv: fix ttl_checker for RawKV API v2.

Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
Signed-off-by: Ping Yu <yuping@pingcap.com>

Co-authored-by: Ping Yu <yuping@pingcap.com>
Signed-off-by: Ping Yu <yuping@pingcap.com>
@ti-chi-bot ti-chi-bot mentioned this issue Jul 19, 2023
Merged
ti-chi-bot pushed a commit to ti-chi-bot/tikv that referenced this issue Jul 19, 2023
@pingyu @ti-chi-bot
This is an automated cherry-pick of tikv#15143
6929aa5 
close tikv#15142

Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
pingyu added a commit that referenced this issue Jul 19, 2023
@ti-chi-bot @pingyu
rawkv: fix ttl_checker for RawKV API v2 (#15143) (#15154) (#15159)
00d952d 
* This is an automated cherry-pick of #15143

close #15142

Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>

* resolve conflict

Signed-off-by: Ping Yu <yuping@pingcap.com>

---------

Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
Signed-off-by: Ping Yu <yuping@pingcap.com>
Co-authored-by: Ping Yu <yuping@pingcap.com>
ti-chi-bot bot added a commit that referenced this issue Jul 24, 2023
@ti-chi-bot @pingyu @ti-chi-bot
rawkv: fix ttl_checker for RawKV API v2 (#15143) (#15155)
8577e53 
close #15142

rawkv: fix ttl_checker for RawKV API v2.

Signed-off-by: Ping Yu <yuping@pingcap.com>

Co-authored-by: Ping Yu <yuping@pingcap.com>
Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pingyu pingyu

Labels
affects-6.5 affects-7.1 severity/major type/bug Type: Issue - Confirmed a bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

rawkv: fix ttl_checker for RawKV API v2 pingyu/tikv
1 participant
@pingyu