Skip to content

Commit 57f7908

Browse files
committed
changefeedccl: replace incorrect usage of QueryRow in tests
Two recently added ALTER CHANGEFEED unit tests use `QueryRow` instead of `Exec` for queries that don't return values, which technically could allow an error to go unnoticed. This patch corrects that. Release note: None
1 parent c18f088 commit 57f7908

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

pkg/ccl/changefeedccl/alter_changefeed_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1826,7 +1826,7 @@ func TestAlterChangefeedAddDropSameTarget(t *testing.T) {
18261826
sqlDB.Exec(t, fmt.Sprintf(`ALTER CHANGEFEED %d ADD bar DROP bar`, feed.JobID()))
18271827
require.NoError(t, feed.Resume())
18281828
var tsStr string
1829-
sqlDB.QueryRow(t, `INSERT INTO bar VALUES(1)`)
1829+
sqlDB.Exec(t, `INSERT INTO bar VALUES(1)`)
18301830
sqlDB.QueryRow(t, `INSERT INTO foo VALUES(2) RETURNING cluster_logical_timestamp()`).Scan(&tsStr)
18311831
ts := parseTimeToHLC(t, tsStr)
18321832
require.NoError(t, feed.WaitForHighWaterMark(ts))

0 commit comments

Comments
 (0)