Skip to content

Commit 02c3e39

Browse files
rustyconoverclaude
andcommitted
fixtures: default batch_size to 2048 (DuckDB STANDARD_VECTOR_SIZE)
CountBatchArgs.batch_size and BATCH_SIZE_FALLBACK defaulted to 1000. Align the canonical batch-size default with DuckDB's default vector size (2048) so the standard sequence-family generators (sequence, double_sequence, nested_sequence, profiling_demo) match the Rust/TypeScript workers, which already default to 2048. Deliberately-varied per-fixture batch sizes are unchanged. Behaviorally invisible to output (batch_size only controls chunking); the one shared SLT that asserts a batch count (dynamic_to_string.test) pins batch_size explicitly, so cross-language workers stay consistent. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent cb125d9 commit 02c3e39

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

‎vgi/_test_fixtures/table/_common.py‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ class CountBatchArgs:
6161
"""
6262

6363
count: Annotated[int, Arg(0, doc="Number of rows to generate", ge=0)]
64-
batch_size: Annotated[int, Arg("batch_size", default=1000, doc="Batch size for output", ge=1)]
64+
batch_size: Annotated[int, Arg("batch_size", default=2048, doc="Batch size for output", ge=1)]
6565

6666

6767
@dataclass(slots=True, frozen=True)
@@ -91,7 +91,7 @@ class _BaseSequenceFunction(TableFunctionGenerator[Any, CountdownState]):
9191
NUMPY_DTYPE: ClassVar[type[np.generic]] = np.int64
9292
STATS_ARROW_TYPE: ClassVar[pa.DataType] = pa.int64()
9393
STATS_COLUMN_NAME: ClassVar[str] = "n"
94-
BATCH_SIZE_FALLBACK: ClassVar[int] = 1000
94+
BATCH_SIZE_FALLBACK: ClassVar[int] = 2048
9595

9696
@classmethod
9797
def initial_state(cls, params: ProcessParams[Any]) -> CountdownState:

0 commit comments

Comments
 (0)