Skip to content

[EPIC] Upstream BigQuery driver patches to ADBC Foundry #16396

Description

@serramatutu

Why

We don't want to drift too much from the upstream because:

  • We want to easily bring in new driver features, patches and bug fixes without many merge conflicts
  • We don't want to maintain duplicate implementations of features that exist both in our fork and upstream

Upstreaming may require changing some code, adding validations, fixing pre-commit and overall cleanup, as their quality standards are generally higher than ours (oops).

End goal

By the end of this epic, our fork should contain minimal differences from upstream, except some dbt-specific hacks that we know won't get accepted upstream (those should be minimal).

Tasks

Done

In-progress

TODO

(NOTE: The PRs show up as merged, but they're merged in our internal repo. I still have to replicate them against upstream).

Out of scope

These will not get accepted upstream. We should come back to them later after the Metadata API revision in ADBC 1.2 (see the discussion in adbc-drivers/bigquery#279)

These PRs are dbt-specific do not need to be upstreamed:

These PRs are backports of what already exists upstream, and when we rebase we should just delete the backport commits:

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

adapter:bigqueryConcerns the BigQuery adapter / SQL dialect.area:adaptersThe adapter framework/layer connecting Fusion to warehouses (dbt-adapter* crates).type:epicA large body of work tracked as an epic spanning multiple issues/PRs.

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions