From c59d66a986cdaf89bd98309c65f02026f2b0e296 Mon Sep 17 00:00:00 2001 From: Nathan VanBenschoten Date: Sat, 29 Apr 2023 21:55:45 -0500 Subject: [PATCH] kv: remove redundant test case in TestTxnCoordSenderRetries This was the same as the "write too old with put after prior read" case. Epic: None Release note: None --- pkg/kv/kvclient/kvcoord/dist_sender_server_test.go | 14 -------------- 1 file changed, 14 deletions(-) diff --git a/pkg/kv/kvclient/kvcoord/dist_sender_server_test.go b/pkg/kv/kvclient/kvcoord/dist_sender_server_test.go index a8cb2bcd44d3..265ef32f1913 100644 --- a/pkg/kv/kvclient/kvcoord/dist_sender_server_test.go +++ b/pkg/kv/kvclient/kvcoord/dist_sender_server_test.go @@ -2434,20 +2434,6 @@ func TestTxnCoordSenderRetries(t *testing.T) { tsLeaked: true, expClientRestart: true, }, - { - name: "write too old with get in the clear", - afterTxnStart: func(ctx context.Context, db *kv.DB) error { - return db.Put(ctx, "a", "put") - }, - retryable: func(ctx context.Context, txn *kv.Txn) error { - if _, err := txn.Get(ctx, "b"); err != nil { - return err - } - return txn.Put(ctx, "a", "put") - }, - expClientRefresh: true, - expClientAutoRetryAfterRefresh: true, - }, { name: "write too old with get conflict", afterTxnStart: func(ctx context.Context, db *kv.DB) error {