kvserver: don't start unrequested verbose logging

Previously in three places inside kvserver we blindly start verbose
logging. With the new raft logging this causes a lot of unnecessary
traces to be logged. This change converts them from verbose to
structured logging.

Epic: none

Release note: None

pkg/kv/kvserver/replica_command.go

@@ -3210,7 +3210,7 @@ func (r *Replica) followerSendSnapshot(
 	if sendThreshold > 0 {
 		var sp *tracing.Span
 		ctx, sp = tracing.EnsureChildSpan(ctx, r.store.cfg.Tracer(),
-			"follower snapshot send", tracing.WithRecording(tracingpb.RecordingVerbose))
+			"follower snapshot send", tracing.WithRecording(tracingpb.RecordingStructured))
 		sendStart := timeutil.Now()
 		defer func() {
 			sendDur := timeutil.Since(sendStart)

pkg/kv/kvserver/replicate_queue.go

@@ -746,7 +746,7 @@ func (rq *replicateQueue) processOneChangeWithTracing(
 ) (requeue bool, _ error) {
 	processStart := timeutil.Now()
 	ctx, sp := tracing.EnsureChildSpan(ctx, rq.Tracer, "process replica",
-		tracing.WithRecording(tracingpb.RecordingVerbose))
+		tracing.WithRecording(tracingpb.RecordingStructured))
 	defer sp.Finish()
 
 	requeue, err := rq.processOneChange(ctx, repl, desc, conf,

pkg/kv/kvserver/split_queue.go

@@ -243,7 +243,7 @@ func (sq *splitQueue) processAttemptWithTracing(
 ) (processed bool, _ error) {
 	processStart := r.Clock().PhysicalTime()
 	ctx, sp := tracing.EnsureChildSpan(ctx, sq.Tracer, "split",
-		tracing.WithRecording(tracingpb.RecordingVerbose))
+		tracing.WithRecording(tracingpb.RecordingStructured))
 	defer sp.Finish()
 
 	processed, err := sq.processAttempt(ctx, r, confReader)