From 6189bb3ce7be13edbc13c47cd6d608246aaf1562 Mon Sep 17 00:00:00 2001 From: Yahor Yuzefovich Date: Thu, 14 Jul 2022 07:47:13 -0700 Subject: [PATCH] roachtest: remove some debugging printouts in tpch_concurrency This was added to track down the deadlock fixed in the previous commit, so we no longer need it. Release note: None --- pkg/cmd/roachtest/tests/tpch_concurrency.go | 17 ----------------- 1 file changed, 17 deletions(-) diff --git a/pkg/cmd/roachtest/tests/tpch_concurrency.go b/pkg/cmd/roachtest/tests/tpch_concurrency.go index 1eb8e8d79b09..02a8b2ddc6e9 100644 --- a/pkg/cmd/roachtest/tests/tpch_concurrency.go +++ b/pkg/cmd/roachtest/tests/tpch_concurrency.go @@ -13,7 +13,6 @@ package tests import ( "context" "fmt" - "strings" "time" "github.com/cockroachdb/cockroach/pkg/cmd/roachtest/cluster" @@ -107,22 +106,6 @@ func registerTPCHConcurrency(r registry.Registry) { // Run each query once on each connection. for queryNum := 1; queryNum <= tpch.NumQueries; queryNum++ { t.Status("running Q", queryNum) - // To aid during the debugging later, we'll print the DistSQL - // diagram of the query. - rows, err := conn.Query("EXPLAIN (DISTSQL) " + tpch.QueriesByNumber[queryNum]) - if err != nil { - return err - } - defer rows.Close() - for rows.Next() { - var line string - if err = rows.Scan(&line); err != nil { - t.Fatal(err) - } - if strings.Contains(line, "Diagram:") { - t.Status(line) - } - } // The way --max-ops flag works is as follows: the global ops // counter is incremented **after** each worker completes a // single operation, so it is possible for all connections start