kvpb: mark subsume as isWrite

Now that subsume is required to force flush all replication streams up
to the lease applied index it froze the range at, it also needs to be
marked as a write in order to pass through the replication code path,
which update the range controller force flush index.

Part of: #132614
Release note: None

pkg/kv/kvnemesis/kvnemesis_test.go

@@ -365,6 +365,14 @@ func testKVNemesisImpl(t *testing.T, cfg kvnemesisTestCfg) {
 	if config.NumReplicas > cfg.numNodes {
 		config.NumReplicas = cfg.numNodes
 	}
+	if cfg.invalidLeaseAppliedIndexProb > 0 {
+		// Merge requests issue a subsume request, which uses the lease applied
+		// index to freeze the range and force flush to the frozen index. Injecting
+		// invalid lease applied indices will cause the merge to fail on the
+		// subsume.
+		config.Ops.Merge.MergeIsSplit = 0
+		config.Ops.Merge.MergeNotSplit = 0
+	}
 	logger := newTBridge(t)
 	env := &Env{SQLDBs: sqlDBs, Tracker: tr, L: logger}
 	failures, err := RunNemesis(ctx, rng, env, config, cfg.concurrency, cfg.numSteps, dbs...)

pkg/kv/kvpb/api.go

@@ -2113,7 +2113,7 @@ func (r *RefreshRangeRequest) flags() flag {
 	return isRead | isTxn | isRange | updatesTSCache
 }
 
-func (*SubsumeRequest) flags() flag    { return isRead | isAlone | updatesTSCache }
+func (*SubsumeRequest) flags() flag    { return isWrite | isAlone | updatesTSCache }
 func (*RangeStatsRequest) flags() flag { return isRead }
 func (*QueryResolvedTimestampRequest) flags() flag {
 	return isRead | isRange | requiresClosedTSOlderThanStorageSnapshot