Skip to content

Conversation

@jiayuasu
Copy link
Member

@jiayuasu jiayuasu commented Jan 19, 2026

This pull request updates the benchmark.yml GitHub Actions workflow to add support for installing and benchmarking against nightly/pre-release builds of SedonaDB and DuckDB. It introduces new workflow inputs and logic to control whether nightly builds are used, making it easier to test against the latest development versions of these dependencies.

Key changes include:

Workflow Inputs and Environment Variables:

  • Added new boolean inputs sedonadb_nightly and duckdb_nightly to the workflow, allowing users to specify whether to use nightly builds for SedonaDB and DuckDB (defaulting to true). Corresponding environment variables are set for use in job steps. [1] [2]

Dependency Installation Logic:

  • Updated the dependency installation steps for both DuckDB and SedonaDB to check the new nightly flags. If enabled, the workflow installs the latest nightly/pre-release builds using the appropriate package indexes and pip flags; otherwise, it falls back to installing the specified version or the latest stable. [1] [2]

Documentation and Debugging:

  • Added echo statements to print out which version and nightly flags are being used for both DuckDB and SedonaDB, improving transparency and debugging in CI logs. [1] [2]

DuckDB Spatial Extension:

  • Clarified that for DuckDB dev builds, the spatial extension is always installed from the default repository, since it's not included in nightly core builds.

@jiayuasu jiayuasu requested a review from Copilot January 20, 2026 03:53
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds support for using nightly/pre-release builds of DuckDB and SedonaDB in the benchmark workflow, allowing testing against the latest development versions of these libraries.

Changes:

  • Added boolean workflow inputs to control nightly build usage for both DuckDB and SedonaDB
  • Modified installation logic to support pre-release builds via pip flags and alternative package indexes
  • Added debug logging to track which installation path is being used

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jiayuasu jiayuasu merged commit ae57d71 into main Jan 20, 2026
14 checks passed
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