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

Resource group name of kv_get request is missing #43499

Closed
Connor1996 opened this issue Apr 28, 2023 · 0 comments · Fixed by #43501
Closed

Resource group name of kv_get request is missing #43499

Connor1996 opened this issue Apr 28, 2023 · 0 comments · Fixed by #43501
Labels
affects-7.1 may-affects-5.1 This bug maybe affects 5.1.x versions. may-affects-5.2 This bug maybe affects 5.2.x versions. may-affects-5.3 This bug maybe affects 5.3.x versions. may-affects-5.4 This bug maybe affects 5.4.x versions. may-affects-6.1 may-affects-6.5 severity/critical type/bug The issue is confirmed as a bug.

Comments

@Connor1996
Copy link
Member

Bug Report

Please answer these questions before submitting your issue. Thanks!

1. Minimal reproduce step (Required)

Run oltp_read_only with two resource group.

2. What did you expect to see? (Required)

The QPS is controlled by RU quota

3. What did you see instead (Required)

The QPS is not controlled by RU quota, and the resource group name of kv_get request is missing
Pl9yB4UbCm

4. What is your TiDB version? (Required)

commit 991d99c
@Connor1996 Connor1996 added the type/bug The issue is confirmed as a bug. label Apr 28, 2023
Connor1996 added a commit to Connor1996/tidb that referenced this issue Apr 28, 2023
@Connor1996
close pingcap#43499 fix missing resource group name for kv_get requests
c88493b 
Signed-off-by: Connor1996 <[email protected]>
@Connor1996 Connor1996 mentioned this issue Apr 28, 2023
Closed
4 tasks
@ti-chi-bot ti-chi-bot bot added may-affects-5.1 This bug maybe affects 5.1.x versions. may-affects-5.2 This bug maybe affects 5.2.x versions. may-affects-5.3 This bug maybe affects 5.3.x versions. may-affects-5.4 This bug maybe affects 5.4.x versions. may-affects-6.1 may-affects-6.5 labels Apr 28, 2023
@Connor1996 Connor1996 mentioned this issue Apr 28, 2023
Merged
4 tasks
@glorv glorv removed may-affects-5.1 This bug maybe affects 5.1.x versions. may-affects-5.2 This bug maybe affects 5.2.x versions. may-affects-5.3 This bug maybe affects 5.3.x versions. may-affects-5.4 This bug maybe affects 5.4.x versions. may-affects-6.1 may-affects-6.5 labels Apr 28, 2023
ti-chi-bot bot pushed a commit that referenced this issue Apr 28, 2023
@Connor1996
resource_control: Fix missing resource group name for kv_get requests (…
275696a 
…#43501)

close #43499
@ti-chi-bot ti-chi-bot mentioned this issue Apr 28, 2023
Merged
4 tasks
@ti-chi-bot ti-chi-bot bot added may-affects-5.1 This bug maybe affects 5.1.x versions. may-affects-5.2 This bug maybe affects 5.2.x versions. may-affects-5.3 This bug maybe affects 5.3.x versions. may-affects-5.4 This bug maybe affects 5.4.x versions. may-affects-6.1 may-affects-6.5 labels Apr 28, 2023
ti-chi-bot bot pushed a commit that referenced this issue May 6, 2023
@ti-chi-bot
resource_control: Fix missing resource group name for kv_get requests (…
f55c7da 
…#43501) (#43502)

close #43499
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
affects-7.1 may-affects-5.1 This bug maybe affects 5.1.x versions. may-affects-5.2 This bug maybe affects 5.2.x versions. may-affects-5.3 This bug maybe affects 5.3.x versions. may-affects-5.4 This bug maybe affects 5.4.x versions. may-affects-6.1 may-affects-6.5 severity/critical type/bug The issue is confirmed as a bug.
Projects
None yet
Milestone
No milestone
3 participants
@glorv @nolouch @Connor1996