Skip to content

Preserve DataFusion benchmark defaults - #50

Merged
gabotechs merged 1 commit into
mainfrom
codex/preserve-datafusion-benchmark-defaults
Aug 14, 2026
Merged

Preserve DataFusion benchmark defaults#50
gabotechs merged 1 commit into
mainfrom
codex/preserve-datafusion-benchmark-defaults

Conversation

@gabotechs

Copy link
Copy Markdown
Owner

Summary

  • make DataFusion session-setting CLI options optional
  • emit each SET statement only when its corresponding flag was explicitly provided
  • preserve explicit false and zero values

This lets benchmark subjects use their own defaults unless the benchmark invocation intentionally overrides them.

Validation

  • npm run build
  • npm test (106 tests)
  • npm --prefix benchmarks-remote run application:build
  • verified CLI help no longer advertises defaults for these flags
  • git diff --check

@gabotechs
gabotechs merged commit 9d2571d into main Aug 14, 2026
2 checks passed
@gabotechs
gabotechs deleted the codex/preserve-datafusion-benchmark-defaults branch August 14, 2026 06:35
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