Skip to content

Build the DataFusion benchmark worker upstream - #47

Merged
gabotechs merged 1 commit into
mainfrom
codex/build-upstream-benchmark-worker
Aug 13, 2026
Merged

Build the DataFusion benchmark worker upstream#47
gabotechs merged 1 commit into
mainfrom
codex/build-upstream-benchmark-worker

Conversation

@gabotechs

Copy link
Copy Markdown
Owner

What changed

  • remove the copied Rust worker crate from benchmarks-remote
  • build datafusion-distributed-benchmarks --bin worker from the checked-out DataFusion Distributed source
  • keep source read-only while allowing each revision to update its cached Cargo.lock
  • use the same standard deployment scripts for humans and the benchmark bot
  • update architecture documentation and tests for the new ownership boundary

Why

The remote worker imports DataFusion Distributed and DataFusion APIs directly. Keeping a separate crate and lockfile in this repository let it drift whenever upstream made a breaking API or dependency change. The worker now lives alongside those APIs and is built from exactly the revision being benchmarked.

Validation

Live result: datafusion-contrib/datafusion-distributed#570 (comment)

@gabotechs
gabotechs merged commit 0f3c780 into main Aug 13, 2026
2 checks passed
@gabotechs
gabotechs deleted the codex/build-upstream-benchmark-worker branch August 13, 2026 10:58
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