sql/logictest: clarify retry comment

Release note: None
cockroachdb · Mar 16, 2023 · 98d1add · 98d1add
1 parent dbbf20d
commit 98d1add
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/pkg/sql/logictest/logic.go b/pkg/sql/logictest/logic.go
@@ -313,8 +313,7 @@ import (
 //            duration. If the test succeeds at any time during that period, it
 //            is considered successful. Otherwise, it is a failure. See
 //            testutils.SucceedsSoon for more information. If run with the
-//            -rewrite flag, inserts a 2s sleep before executing the query
-//            once.
+//            -rewrite flag, the query will be run only once after a 2s sleep.
 //      - async: runs the query asynchronously, marking it as a pending
 //            query using the label parameter as a unique name, to be completed
 //            and validated later with "awaitquery". This is intended for use

diff --git a/pkg/sql/logictest/testdata/logic_test/retry b/pkg/sql/logictest/testdata/logic_test/retry
@@ -4,6 +4,8 @@ SELECT nextval('s')
 
 # Unit test for bug #95664, retry on a query producing an error should
 # try again and succeed if err doesn't happen again.
+# NOTE: This test will fail when the --rewrite flag is used because the query
+# will only be run once.
 query I retry
 SELECT
   CASE subq.val