Skip to content

Commit

Permalink
sql: skip reset sql stats in TestRandomSyntaxFunctions
Browse files Browse the repository at this point in the history
Previously, crdb_internal.reset_sql_stats() causes timeout
in TestRandomSyntaxFunctions. This is very unlikely due to
implementation of the function, and it is likely caused
by contentions.
This commit skip the tests for crdb_internal.reset_sql_stats()
to prevent nightly failures.

Related cockroachdb#69731

Release justification: Non-production code changes

Release note: None
  • Loading branch information
Azhng committed Sep 9, 2021
1 parent 683eae2 commit 8ef68b8
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions pkg/sql/tests/rsg_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -298,6 +298,9 @@ func TestRandomSyntaxFunctions(t *testing.T) {
// Calculating the Frechet distance is slow and testing it here
// is not worth it.
continue
case "crdb_internal.reset_sql_stats":
// Skipped due to long execution time.
continue
}
_, variations := builtins.GetBuiltinProperties(name)
for _, builtin := range variations {
Expand Down

0 comments on commit 8ef68b8

Please sign in to comment.