Skip to content

Add wide_schema to standard benchmark allowlist#10

Merged
adriangb merged 1 commit into
mainfrom
add-wide-schema-criterion
Jun 8, 2026
Merged

Add wide_schema to standard benchmark allowlist#10
adriangb merged 1 commit into
mainfrom
add-wide-schema-criterion

Conversation

@adriangb

@adriangb adriangb commented Jun 8, 2026

Copy link
Copy Markdown
Owner

Summary

wide_schema is a bench.sh standard benchmark added upstream in apache/datafusion#21970 (bench.sh data/run wide_schema, backed by a gen_wide_data binary and a sql_benchmarks/wide_schema query set — 1024-col × 256-file wide dataset vs an 8-col narrow baseline).

It was missing from the standard allowlist, so run benchmark wide_schema (e.g. on apache/datafusion#22829) fell through to the wildcard criterion path and ran cargo bench --bench wide_schema — which has no such target — and failed with cargo's "available bench targets" dump.

Changes

  • Add wide_schema to the standard list for apache/datafusion and the adriangb/datafusion test fork in services/controller.ts, routing it to the bench.sh standard runner.
  • Document it in the README standard benchmarks table.

🤖 Generated with Claude Code

wide_schema is a bench.sh standard benchmark (added upstream in
apache/datafusion#21970): `bench.sh data/run wide_schema`, backed by a
gen_wide_data binary and sql_benchmarks/wide_schema query set.

It was missing from the standard allowlist, so `run benchmark
wide_schema` fell through to the wildcard criterion path and ran
`cargo bench --bench wide_schema`, which has no such target and failed.
Adding it to the standard list routes it to the bench.sh runner.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@adriangb adriangb merged commit 33993f7 into main Jun 8, 2026
3 checks passed
@adriangb adriangb deleted the add-wide-schema-criterion branch June 8, 2026 19:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant