Skip to content

resource control: support more mode for BURSTABLE#20020

Merged
ti-chi-bot[bot] merged 28 commits intopingcap:masterfrom
lhy1024:burstable
Aug 19, 2025
Merged

resource control: support more mode for BURSTABLE#20020
ti-chi-bot[bot] merged 28 commits intopingcap:masterfrom
lhy1024:burstable

Conversation

@lhy1024
Copy link
Copy Markdown
Contributor

@lhy1024 lhy1024 commented Mar 7, 2025

First-time contributors' checklist

What is changed, added or deleted? (Required)

Which TiDB version(s) do your changes apply to? (Required)

Tips for choosing the affected version(s):

By default, CHOOSE MASTER ONLY so your changes will be applied to the next TiDB major or minor releases. If your PR involves a product feature behavior change or a compatibility change, CHOOSE THE AFFECTED RELEASE BRANCH(ES) AND MASTER.

For details, see tips for choosing the affected versions (in Chinese).

  • master (the latest development version)
  • v9.0 (TiDB 9.0 versions)
  • v8.5 (TiDB 8.5 versions)
  • v8.4 (TiDB 8.4 versions)
  • v8.3 (TiDB 8.3 versions)
  • v8.1 (TiDB 8.1 versions)
  • v7.5 (TiDB 7.5 versions)
  • v7.1 (TiDB 7.1 versions)
  • v6.5 (TiDB 6.5 versions)
  • v6.1 (TiDB 6.1 versions)
  • v5.4 (TiDB 5.4 versions)

What is the related PR or file link(s)?

  • This PR is translated from:
  • Other reference link(s):

Do your changes match any of the following descriptions?

  • Delete files
  • Change aliases
  • Need modification after applied to another branch
  • Might cause conflicts after applied to another branch

lhy1024 added 2 commits March 7, 2025 02:18
Signed-off-by: lhy1024 <admin@liudos.us>
Signed-off-by: lhy1024 <admin@liudos.us>
@ti-chi-bot ti-chi-bot bot added missing-translation-status This PR does not have translation status info. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Mar 7, 2025
@songrijie songrijie self-requested a review March 7, 2025 02:48
Signed-off-by: lhy1024 <admin@liudos.us>
@hfxsd hfxsd added translation/doing This PR’s assignee is translating this PR. v9.0-beta.1 This PR/issue applies to TiDB v9.0-beta.1. and removed missing-translation-status This PR does not have translation status info. labels Mar 10, 2025
@hfxsd hfxsd self-assigned this Mar 10, 2025
lhy1024 and others added 3 commits March 12, 2025 14:47
Co-authored-by: Roger Song <roger.song@pingcap.com>
Co-authored-by: Roger Song <roger.song@pingcap.com>
Co-authored-by: Roger Song <roger.song@pingcap.com>
@ti-chi-bot
Copy link
Copy Markdown

ti-chi-bot bot commented Mar 12, 2025

@songrijie: adding LGTM is restricted to approvers and reviewers in OWNERS files.

Details

In response to this:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@hfxsd hfxsd added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. and removed v9.0-beta.1 This PR/issue applies to TiDB v9.0-beta.1. labels Mar 18, 2025
@hfxsd hfxsd added the v9.0-beta.2 This PR/issue applies to TiDB v9.0-beta.2. label Mar 31, 2025
@lhy1024
Copy link
Copy Markdown
Contributor Author

lhy1024 commented Apr 27, 2025

@hfxsd ptal

@lilin90 lilin90 assigned lilin90 and unassigned hfxsd May 29, 2025
@lilin90 lilin90 requested review from hfxsd and lilin90 May 29, 2025 07:53
@lilin90 lilin90 removed the translation/doing This PR’s assignee is translating this PR. label Jun 11, 2025
Copy link
Copy Markdown
Member

@lilin90 lilin90 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@ti-chi-bot ti-chi-bot bot added the needs-1-more-lgtm Indicates a PR needs 1 more LGTM. label Jun 27, 2025
+---------+------------+----------+----------------+-------------+------------+
| NAME | RU_PER_SEC | PRIORITY | BURSTABLE | QUERY_LIMIT | BACKGROUND |
+---------+------------+----------+----------------+-------------+------------+
| default | UNLIMITED | MEDIUM | YES(UNLIMITED) | NULL | NULL |
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里 BURSTABLE 是不是就只显示 OFF, UNLIMITED, MODERATED。 用括号不太合适。 @lhy1024

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

要修改显示结果么?不过不知道代码还能不能合进去了

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

修改一下吧, 这个不太符合系统表的习惯。

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ti-chi-bot ti-chi-bot bot added lgtm and removed needs-1-more-lgtm Indicates a PR needs 1 more LGTM. labels Jul 3, 2025
@ti-chi-bot
Copy link
Copy Markdown

ti-chi-bot bot commented Jul 3, 2025

[LGTM Timeline notifier]

Timeline:

  • 2025-06-27 03:00:10.607266052 +0000 UTC m=+1018263.330445035: ☑️ agreed by lilin90.
  • 2025-07-03 02:32:56.513322959 +0000 UTC m=+1535029.236501936: ☑️ agreed by hfxsd.

lhy1024 and others added 4 commits July 3, 2025 10:55
Co-authored-by: xixirangrang <hfxsd@hotmail.com>
Signed-off-by: lhy1024 <admin@liudos.us>
Co-authored-by: xixirangrang <hfxsd@hotmail.com>
@lilin90
Copy link
Copy Markdown
Member

lilin90 commented Jul 14, 2025

This PR is ready for merging.

@lilin90
Copy link
Copy Markdown
Member

lilin90 commented Aug 19, 2025

/unhold

@ti-chi-bot ti-chi-bot bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Aug 19, 2025
Copy link
Copy Markdown
Member

@lilin90 lilin90 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/approve

@ti-chi-bot
Copy link
Copy Markdown

ti-chi-bot bot commented Aug 19, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lilin90

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ti-chi-bot ti-chi-bot bot added the approved label Aug 19, 2025
@ti-chi-bot ti-chi-bot bot merged commit e2077e8 into pingcap:master Aug 19, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved lgtm size/L Denotes a PR that changes 100-499 lines, ignoring generated files. translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR. v9.0-beta.2 This PR/issue applies to TiDB v9.0-beta.2.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants