Merge #98006

98006: copy: fix data race in test r=otan a=rafiss The test code had two goroutines that were both trying to assign to the same `err` variable. fixes #97838 Release note: None Co-authored-by: Rafi Shamim <[email protected]>
cockroachdb · Mar 6, 2023 · 1d4feea · 1d4feea
2 parents c6996f0 + ff43bd8
commit 1d4feea
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/pkg/sql/copy/copy_test.go b/pkg/sql/copy/copy_test.go
@@ -412,7 +412,7 @@ func TestShowQueriesIncludesCopy(t *testing.T) {
 	t.Run("copy to", func(t *testing.T) {
 		g := ctxgroup.WithContext(ctx)
 		g.GoCtx(func(ctx context.Context) error {
-			_, err = copyConn.Exec(ctx, "COPY (SELECT pg_sleep(1) FROM ROWS FROM (generate_series(1, 60)) AS i) TO STDOUT")
+			_, err := copyConn.Exec(ctx, "COPY (SELECT pg_sleep(1) FROM ROWS FROM (generate_series(1, 60)) AS i) TO STDOUT")
 			return err
 		})