Skip to content

coprocessor: backport paging limits to release-nextgen-202603#1501

Merged
ti-chi-bot[bot] merged 2 commits into
pingcap:release-nextgen-202603from
JmPotato:codex/paging-size-bytes-release-nextgen-202603
Jul 16, 2026
Merged

coprocessor: backport paging limits to release-nextgen-202603#1501
ti-chi-bot[bot] merged 2 commits into
pingcap:release-nextgen-202603from
JmPotato:codex/paging-size-bytes-release-nextgen-202603

Conversation

@JmPotato

Copy link
Copy Markdown
Member

What changed

Backport the coprocessor request fields required by byte-budget paging to release-nextgen-202603:

Both fields are optional uint64 fields. A zero value keeps the existing behavior.

Why

The TiDB Resource Control paging backport needs coprocessor.Request.paging_size_bytes while remaining on the kvproto release-nextgen-202603 lineage instead of consuming a master-only commit.

Related downstream PRs:

Source commits

Each source commit is preserved as a separate signed-off cherry-pick with provenance.

Tests

make check
make go
go build ./pkg/...
git diff --check upstream/release-nextgen-202603...HEAD

Tema and others added 2 commits July 16, 2026 18:27
Signed-off-by: artem_danilov <artem.danilov@airbnb.com>
(cherry picked from commit bb32227)
Signed-off-by: JmPotato <github@ipotato.me>
Add a new uint64 field `paging_size_bytes` (field 17) to the
coprocessor Request message. When set, TiKV will stop scanning
when accumulated scanned bytes reach this limit, enabling
byte-budget paging for finer-grained Resource Control.

Note: rebased onto upstream master where field 16 is now taken
by max_keys_read (pingcap#1450), so paging_size_bytes was bumped from
field 16 to field 17.

Signed-off-by: JmPotato <github@ipotato.me>
(cherry picked from commit a15c348)
Signed-off-by: JmPotato <github@ipotato.me>
@ti-chi-bot ti-chi-bot Bot requested review from Tristan1900 and mornyx July 16, 2026 10:31
@ti-chi-bot ti-chi-bot Bot added the size/XL label Jul 16, 2026
@ti-chi-bot ti-chi-bot Bot added the lgtm label Jul 16, 2026
@ti-chi-bot

ti-chi-bot Bot commented Jul 16, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cfzjywxk, rleungx

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 commented Jul 16, 2026

Copy link
Copy Markdown

[LGTM Timeline notifier]

Timeline:

  • 2026-07-16 10:54:22.965538113 +0000 UTC m=+883849.001633159: ☑️ agreed by cfzjywxk.
  • 2026-07-16 11:09:51.438374531 +0000 UTC m=+884777.474469587: ☑️ agreed by rleungx.

@ti-chi-bot ti-chi-bot Bot merged commit 3631780 into pingcap:release-nextgen-202603 Jul 16, 2026
2 checks passed
@JmPotato JmPotato deleted the codex/paging-size-bytes-release-nextgen-202603 branch July 16, 2026 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants