Skip to content

Commit

Permalink
roachtest: fix a minor bug with tpch_concurrency roachtest
Browse files Browse the repository at this point in the history
`tpch_concurrency` pushes the cluster to its limits, so it is expected
that OOMs occur. Previously, in a recently added code path, we would
fail the test if an error occurred when running a debugging statement,
yet that error is expected and should be returned.

Release note: None
  • Loading branch information
yuzefovich committed Jun 7, 2022
1 parent 7377f2d commit 01cfeb0
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/cmd/roachtest/tests/tpch_concurrency.go
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,7 @@ func registerTPCHConcurrency(r registry.Registry) {
// diagram of the query.
rows, err := conn.Query("EXPLAIN (DISTSQL) " + tpch.QueriesByNumber[queryNum])
if err != nil {
t.Fatal(err)
return err
}
defer rows.Close()
for rows.Next() {
Expand Down

0 comments on commit 01cfeb0

Please sign in to comment.