Skip to content

Commit 2ce34f8

Browse files
antiguruclaude
andcommitted
test: reset enable_compute_peek_row_iteration_limit after use
persist-fast-path.slt and max_result_size.slt reset compute_peek_row_iteration_limit at the end but leave the feature flag itself enabled. That is harmless under bin/sqllogictest, whose suite default for the limit is very high, but wrong against a plain environmentd where the code default is 1000. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
1 parent f3e7a75 commit 2ce34f8

2 files changed

Lines changed: 10 additions & 0 deletions

File tree

‎test/sqllogictest/max_result_size.slt‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -86,6 +86,11 @@ ALTER SYSTEM RESET compute_peek_row_iteration_limit;
8686
----
8787
COMPLETE 0
8888

89+
simple conn=mz_system,user=mz_system
90+
ALTER SYSTEM RESET enable_compute_peek_row_iteration_limit;
91+
----
92+
COMPLETE 0
93+
8994
statement ok
9095
SET cluster TO 'quickstart';
9196

‎test/sqllogictest/persist-fast-path.slt‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -275,6 +275,11 @@ ALTER SYSTEM RESET compute_peek_row_iteration_limit;
275275
----
276276
COMPLETE 0
277277

278+
simple conn=mz_system,user=mz_system
279+
ALTER SYSTEM RESET enable_compute_peek_row_iteration_limit;
280+
----
281+
COMPLETE 0
282+
278283
# The Persist fast path does not apply when an index exists.
279284

280285
statement ok

0 commit comments

Comments
 (0)