sql: make RETURNING NOTHING statements not cause FirstBatch->Open tra…

…nsition

The parallelized statements don't return results to the client, by
definition. This commit adds them to the list of statements that don't
transition the session from the FirstBatch to Open, so that the next
batch of statements can be auto-retried.

pkg/sql/executor.go

@@ -2015,6 +2015,8 @@ func canStayInFirstBatchState(stmt Statement) bool {
 	return isBegin(stmt) ||
 		isSavepoint(stmt) ||
 		isSetTransaction(stmt) ||
+		// Parallelized statements don't return results to clients, by definition.
+		IsStmtParallelized(stmt) ||
 		// ROLLBACK TO SAVEPOINT does its own state transitions; if it leave the
 		// transaction in the FirstBatch state, don't mess with it.
 		isRollbackToSavepoint(stmt)

pkg/sql/logictest/testdata/logic_test/txn

@@ -624,6 +624,18 @@ statement ok
 ROLLBACK
 
 
+# Parallelized statements doesn't prevent automatic retries of the first batch.
+# Restore when #16827 is fixed.
+# statement ok
+# BEGIN TRANSACTION; INSERT INTO kv (k,v) VALUES ('x', 'b') RETURNING NOTHING;
+#
+# statement ok
+# SELECT 1; SELECT CRDB_INTERNAL.FORCE_RETRY('100ms':::INTERVAL)
+#
+# statement ok
+# ROLLBACK
+
+
 # General savepoints
 statement ok
 BEGIN TRANSACTION