Merge #152394

craig[bot] · rafiss · craig[bot] · commit 4582d771cefc · 2025-08-25T18:17:07.000Z
152394: schemachanger: don't run TestAlterTableDMLInjection with t.Parallel r=rafiss a=rafiss Under deadlock/race configurations, this can add too much load to the test runner. fixes #152254 fixes #152339 fixes #152258 fixes #152255 fixes #152256 fixes #152252 fixes #152411 Release note: None Co-authored-by: Rafi Shamim <rafi@cockroachlabs.com>
diff --git a/pkg/sql/schemachanger/dml_injection_test.go b/pkg/sql/schemachanger/dml_injection_test.go
@@ -563,7 +563,6 @@ func TestAlterTableDMLInjection(t *testing.T) {
 	ctx := context.Background()
 	for _, tc := range testCases {
 		t.Run(tc.desc, tc.capture(func(t *testing.T, tc testCase) {
-			t.Parallel() // SAFE FOR TESTING
 			if issue := tc.skipIssue; issue != 0 {
 				skip.WithIssue(t, issue)
 			}

Original file line number	Diff line number	Diff line change
`@@ -563,7 +563,6 @@ func TestAlterTableDMLInjection(t *testing.T) {`
`563`	`563`	`ctx := context.Background()`
`564`	`564`	`for _, tc := range testCases {`
`565`	`565`	`t.Run(tc.desc, tc.capture(func(t *testing.T, tc testCase) {`
`566`		`- t.Parallel() // SAFE FOR TESTING`
`567`	`566`	`if issue := tc.skipIssue; issue != 0 {`
`568`	`567`	`skip.WithIssue(t, issue)`
`569`	`568`	`}`