Merge #146589

146589: roachtest: increase size of schemachange/bulkingest r=rafiss a=rafiss

We are starting to test index backfills at larger scales. Let's make this test bigger so we can more easily track improvements over time.

informs #146571
Release note: None

Co-authored-by: Rafi Shamim <[email protected]>

Author: craig[bot]

pkg/cmd/roachtest/tests/schemachange.go

@@ -330,7 +330,7 @@ func makeIndexAddTpccTest(
 func registerSchemaChangeBulkIngest(r registry.Registry) {
 	// Allow a long running time to account for runs that use a
 	// cockroach build with runtime assertions enabled.
-	r.Add(makeSchemaChangeBulkIngestTest(r, 5, 100000000, time.Minute*60))
+	r.Add(makeSchemaChangeBulkIngestTest(r, 12, 4_000_000_000, 2*time.Hour))
 }
 
 func makeSchemaChangeBulkIngestTest(
@@ -346,18 +346,20 @@ func makeSchemaChangeBulkIngestTest(
 		Timeout:          length * 2,
 		// `fixtures import` (with the workload paths) is not supported in 2.1
 		Run: func(ctx context.Context, t test.Test, c cluster.Cluster) {
-			// Configure column a to have sequential ascending values, and columns b and c to be constant.
-			// The payload column will be randomized and thus uncorrelated with the primary key (a, b, c).
-			aNum := numRows
+			// Configure column a to have sequential ascending values. The payload
+			// column will be randomized and thus uncorrelated with the primary key
+			// (a, b, c).
+			bNum := 1000
+			cNum := 1000
+			aNum := numRows / (bNum * cNum)
 			if c.IsLocal() {
 				aNum = 100000
+				bNum = 1
+				cNum = 1
 			}
-			bNum := 1
-			cNum := 1
-			payloadBytes := 4
+			payloadBytes := 40
 
-			// TODO (lucy): Remove flag once the faster import is enabled by default
-			settings := install.MakeClusterSettings(install.EnvOption([]string{"COCKROACH_IMPORT_WORKLOAD_FASTER=true"}))
+			settings := install.MakeClusterSettings()
 			c.Start(ctx, t.L(), option.DefaultStartOpts(), settings, c.CRDBNodes())
 
 			// Don't add another index when importing.