client: support versioned coprocessor routing

[wshwsh12]

• route version-aware cop to VersionedKv
• add versioned cop cmd types in tikvrpc

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

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

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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 and usage tips.}

[ChangRui-Ryan]

Suggest putting CmdVersionedCop right after CmdCop wherever it appears to keep consistency, except CmdVersionedCop is at the end of the CmdType enum.

[ChangRui-Ryan]

lgtm

[cfzjywxk]

BatchReq would be used by default at L354, I suppose this may not be reached, have you executed E2E test for this PR?

[wshwsh12]

"ToBatchCommandsRequest" can't converts CmdVersionedCop to BatchReq and return nil. So now CmdVersionedCop always reached this path.
And I have run some sqls in my local and it works.

[you06]

Here are some performance risks. How often is this CmdVersionedCop called? If the frequency is high, I think it's necessary to modify kvproto to include it in the BatchReq and BatchResp.

[wshwsh12]

Currently, this feature does not have high performance requirements, and RPC requests are not the performance bottleneck. We can optimize it later if performance becomes an issue.”

[you06]

isReadReq needs to be updated.

client-go/internal/locate/replica_selector.go

Lines 605 to 613 in 6e80a22

	func isReadReq(tp tikvrpc.CmdType) bool {
	switch tp {
	case tikvrpc.CmdGet, tikvrpc.CmdBatchGet, tikvrpc.CmdScan,
	tikvrpc.CmdCop, tikvrpc.CmdBatchCop, tikvrpc.CmdCopStream:
	return true
	default:
	return false
	}
	}

[zyguan]

Should we handle CmdVersionedCop in codec v2 ?

[wshwsh12]

Should we handle CmdVersionedCop in codec v2 ?

My understanding is that only keyspace requires codec v2. Currently, VersionedCop has not been implemented for keyspace. When I need to implement VersionedCop for keyspace, I will add the corresponding support in client-go.

[zyguan]

My understanding is that only keyspace requires codec v2. Currently, VersionedCop has not been implemented for keyspace. When I need to implement VersionedCop for keyspace, I will add the corresponding support in client-go.

So, how about to add the following code to codec v2?

	case tikvrpc.CmdVersionedCop:
		// TODO: ...
		return nil, errors.New("...")

[ti-chi-bot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: you06, zyguan

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 [you06,zyguan]
• internal/client/OWNERS [you06,zyguan]

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]

[LGTM Timeline notifier]

Timeline:

• 2026-02-13 02:31:25.668810651 +0000 UTC m=+158438.748801356: ☑️ agreed by zyguan.
• 2026-02-13 07:25:20.344725056 +0000 UTC m=+176073.424715761: ☑️ agreed by you06.