Skip to content

Comments

Add BigQuery reservation configuration#2096

Open
max-ostapenko wants to merge 1 commit intodataform-co:mainfrom
max-ostapenko:witty-lynx
Open

Add BigQuery reservation configuration#2096
max-ostapenko wants to merge 1 commit intodataform-co:mainfrom
max-ostapenko:witty-lynx

Conversation

@max-ostapenko
Copy link

@max-ostapenko max-ostapenko commented Feb 24, 2026

This pull request introduces support for specifying BigQuery reservations at both the project and action level, allowing more granular control over query resource allocation. The changes span CLI flags, project configuration, action builders, and test coverage to ensure correct application and precedence of reservation settings.

BigQuery reservation support:

  • Added a new --default-reservation CLI flag and corresponding defaultReservation project config option, enabling users to set a default BigQuery reservation for all actions in a project.
  • Updated IBigQueryExecutionOptions and BigQueryDbAdapter to accept and propagate a reservation parameter, ensuring reservation information is passed through to BigQuery job execution.

Action-level reservation configuration:

  • Extended action builders (Assertion, Table, IncrementalTable, Operation, View) to support an optional bigqueryReservation config property, storing it in the action descriptor for runtime precedence over project-level reservations.

Test coverage and validation:

  • Added and updated tests to verify correct application of project-level and action-level reservations, including precedence rules and propagation to compiled outputs.

Project configuration precedence:

  • Ensured that action-level bigqueryReservation overrides the project-level defaultReservation at runtime, with tests confirming the correct behavior.

These changes provide users with flexible reservation management for Dataform workload.

Related to #1981

@max-ostapenko max-ostapenko requested a review from a team as a code owner February 24, 2026 21:13
@max-ostapenko max-ostapenko requested review from Ceridan and removed request for a team February 24, 2026 21:13
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