resource_usage_agent: add RocksDB block read count to Top SQL resource dimensions

[jiong-nba]

What

Add the rocksdb_block_read_count field to GroupTagRecordItem so TiKV can report foreground RocksDB block reads through resource metering.

Issue Number: ref #1507

Ref: tikv/tikv#19867

Validation

• make check
• make go

Summary by CodeRabbit

• New Features
  • Added a RocksDB block-read count metric to resource usage records.
  • Enables more accurate attribution of read I/O activity to downstream workloads.
  • Clarifies that this metric represents RocksDB block reads rather than device-level IOPS.

[pingcap-cla-assistant]

All committers have signed the CLA.

[Copilot]

Pull request overview

This PR extends the resource usage metering protobuf schema to allow TiKV to report foreground RocksDB block read counts as part of group-tag resource metering records.

Changes:

• Added a new rocksdb_block_read_count field to GroupTagRecordItem in resource_usage_agent.proto (field number 9, uint64).
• Updated the generated Go bindings to include the new field, plus marshal/size/unmarshal support.
• Updated scripts/proto.lock to reflect the schema change.

Reviewed changes

Copilot reviewed 1 out of 3 changed files in this pull request and generated no comments.

File	Description
scripts/proto.lock	Updates the locked proto schema metadata to include the new field in GroupTagRecordItem.
proto/resource_usage_agent.proto	Adds rocksdb_block_read_count to GroupTagRecordItem with tag 9.
pkg/resource_usage_agent/resource_usage_agent.pb.go	Regenerates Go protobuf code to include the new field and its serialization logic.

Files not reviewed (1)

• pkg/resource_usage_agent/resource_usage_agent.pb.go: Generated file

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[coderabbitai]

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)

• scripts/proto.lock is excluded by !**/*.lock

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 9622d473-1e74-471e-8066-d8f7a58e5b86

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

📝 Walkthrough

Walkthrough

The resource usage protobuf adds rocksdb_block_read_count to GroupTagRecordItem as field 9. Its documentation identifies the value as a RocksDB block-read count, not a device-level IOPS measurement.

Changes

Resource usage reporting

Layer / File(s)	Summary
Add RocksDB block-read count proto/resource_usage_agent.proto	GroupTagRecordItem now includes the uint64 rocksdb_block_read_count = 9 field with documentation for downstream read-I/O attribution.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related issues

• resource_usage_agent: add RocksDB block read count to Top SQL resource dimensions #1507 — Adds the same rocksdb_block_read_count field to GroupTagRecordItem.

Suggested reviewers: xzhangxian1008

Poem

A rabbit counts blocks in the stream,
Field nine records each read-I/O beam.
RocksDB reads are clear and bright,
Separate from device IOPS in sight.
One small field completes the scheme.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly identifies the addition of RocksDB block read counts to the Top SQL resource dimensions.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch codex/add-resource-metering-block-read-count

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

Note

GitHub couldn't provide a complete incremental comparison for this pull request, so CodeRabbit is performing a full review instead. This review may take a little longer.

[yibin87]

LGTM

[ti-chi-bot]

[LGTM Timeline notifier]

Timeline:

• 2026-08-13 01:58:05.677329455 +0000 UTC m=+3270871.713424501: ☑️ agreed by yibin87.
• 2026-08-13 04:27:34.302285976 +0000 UTC m=+3279840.338381032: ☑️ agreed by nolouch.

[jiong-nba]

/retest

[jiong-nba]

/retest

[ti-chi-bot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: nolouch, XuHuaiyu, yibin87

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:

• OWNERS [XuHuaiyu,nolouch]
• proto/OWNERS [XuHuaiyu]

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