kvserver: remove StoreBenignError

Before commit 3f0b37a, the
StoreBenignError is used to handle pebble.ErrSnapshotExcised. As the
latter has been removed from pebble, we don't need StoreBenignError
anymore.

This commit does the following:

- Remove type "StoreBenignError".
- Remove the related increase action on counter "storeFailures".
- Update related tests "TestBaseQueueRequeue".

Fixes: #129941

Release note: None

Author: XiaochenCui

Diff for: pkg/kv/kvserver/benignerror/BUILD.bazel

@@ -2,10 +2,7 @@ load("@io_bazel_rules_go//go:def.bzl", "go_library")
 
 go_library(
     name = "benignerror",
-    srcs = [
-        "benign_error.go",
-        "store_benign_error.go",
-    ],
+    srcs = ["benign_error.go"],
     importpath = "github.com/cockroachdb/cockroach/pkg/kv/kvserver/benignerror",
     visibility = ["//visibility:public"],
     deps = ["@com_github_cockroachdb_errors//:errors"],

Diff for: pkg/kv/kvserver/benignerror/store_benign_error.go

@@ -1170,17 +1170,12 @@ func (bq *baseQueue) finishProcessingReplica(
 	// Handle failures.
 	if err != nil {
 		benign := benignerror.IsBenign(err)
-		storeBenign := benignerror.IsStoreBenign(err)
 
 		// Increment failures metric.
 		//
 		// TODO(tschottdorf): once we start asserting zero failures in tests
 		// (and production), move benign failures into a dedicated category.
 		bq.failures.Inc(1)
-		if storeBenign {
-			bq.storeFailures.Inc(1)
-			requeue = true
-		}
 
 		// Determine whether a failure is a purgatory error. If it is, add
 		// the failing replica to purgatory. Note that even if the item was

Diff for: pkg/kv/kvserver/queue.go

@@ -18,7 +18,6 @@ import (
 	"github.com/cockroachdb/cockroach/pkg/config"
 	"github.com/cockroachdb/cockroach/pkg/config/zonepb"
 	"github.com/cockroachdb/cockroach/pkg/keys"
-	"github.com/cockroachdb/cockroach/pkg/kv/kvserver/benignerror"
 	"github.com/cockroachdb/cockroach/pkg/kv/kvserver/kvserverbase"
 	"github.com/cockroachdb/cockroach/pkg/roachpb"
 	"github.com/cockroachdb/cockroach/pkg/settings"
@@ -1559,18 +1558,4 @@ func TestBaseQueueRequeue(t *testing.T) {
 	bq.maybeAdd(ctx, r1, hlc.ClockTimestamp{})
 	assertShouldQueueCount(6)
 	assertProcessedAndProcessing(2, 0)
-
-	// Reset shouldQueueCount so we actually process the replica. Then return
-	// a StoreBenign error. It should requeue the replica.
-	atomic.StoreInt64(&shouldQueueCount, 0)
-	pQueue.err = benignerror.NewStoreBenign(errors.New("test"))
-	bq.maybeAdd(ctx, r1, hlc.ClockTimestamp{})
-	assertShouldQueueCount(1)
-	assertProcessedAndProcessing(2, 1)
-	// Let the first processing attempt finish. It should requeue.
-	pQueue.processBlocker <- struct{}{}
-	assertProcessedAndProcessing(3, 1)
-	pQueue.err = nil
-	pQueue.processBlocker <- struct{}{}
-	assertProcessedAndProcessing(4, 0)
 }