diff --git a/pkg/storage/intent_resolver.go b/pkg/storage/intent_resolver.go
index 94ccd90cdc3e..69bbf47b2ba6 100644
--- a/pkg/storage/intent_resolver.go
+++ b/pkg/storage/intent_resolver.go
@@ -105,8 +105,18 @@ func (ir *intentResolver) processWriteIntentError(
 		return pErr
 	}
 
+	// We always poison due to limitations of the API: not poisoning equals
+	// clearing the abort cache, and if our pushee transaction first got pushed
+	// for timestamp (by us), then (by someone else) aborted and poisoned, and
+	// then we run the below code, we're clearing the abort cache illegaly.
+	// Furthermore, even if our pushType is not PUSH_ABORT, we may have ended
+	// up with the responsiblity to abort the intents (for example if we find
+	// the transaction aborted).
+	//
+	// To do better here, we need per-intent information on whether we need to
+	// poison.
 	if err := ir.resolveIntents(ctx, resolveIntents,
-		ResolveOptions{Wait: false, Poison: pushType == roachpb.PUSH_ABORT}); err != nil {
+		ResolveOptions{Wait: false, Poison: true}); err != nil {
 		return roachpb.NewError(err)
 	}
 
diff --git a/pkg/storage/replica_test.go b/pkg/storage/replica_test.go
index 8c1662b7d9e4..9b5312eae360 100644
--- a/pkg/storage/replica_test.go
+++ b/pkg/storage/replica_test.go
@@ -4877,7 +4877,7 @@ func TestReplicaResolveIntentNoWait(t *testing.T) {
 			Span:   roachpb.Span{Key: key},
 			Txn:    txn.TxnMeta,
 			Status: txn.Status,
-		}}, ResolveOptions{Wait: false, Poison: false}); pErr != nil {
+		}}, ResolveOptions{Wait: false, Poison: true /* irrelevant */}); pErr != nil {
 		t.Fatal(pErr)
 	}
 	testutils.SucceedsSoon(t, func() error {