Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ADAP-1128: Migrate dbt-postgres into dbt-adapters #615

Merged
merged 120 commits into from
Jan 24, 2025

Conversation

mikealfare
Copy link
Contributor

Migrate dbt-postgres into this repo using the following shell script and call:

repo=$1
source_branch=$2
target_branch=$3

# create a remote for the legacy adapter repo and fetch the latest commits
git remote remove old || true
git remote add old https://github.com/dbt-labs/$repo.git
git fetch old

# merge the updated branch from the legacy repo into the dbt-adapters repo
git checkout $target_branch
git merge old/$source_branch --allow-unrelated-histories

# remove the remote that was created by this process
git remote remove old || true
source ./scripts/migrate-adapter.sh dbt-postgres monorepo/stage monorepo/migrate-dbt-postgres

Update workflows to include dbt-postgres.

Make updates related with code quality checks due to slightly different config.

Merge this PR as a regular merge instead of a squash merge in order to retain git blame history.

@mikealfare
Copy link
Contributor Author

Integration tests run against this branch: https://github.com/dbt-labs/dbt-adapters/actions/runs/12920407631/job/36032667661

@mikealfare mikealfare marked this pull request as ready for review January 23, 2025 01:39
@mikealfare mikealfare requested a review from a team as a code owner January 23, 2025 01:39
@mikealfare
Copy link
Contributor Author

The dbt-redshift failures are all the IAM Role env variable issue and the dbt-athena failures are the iceberg retries issue. These are known issues and unrelated to dbt-postgres.

@mikealfare mikealfare merged commit 8b303eb into main Jan 24, 2025
211 of 218 checks passed
@mikealfare mikealfare deleted the monorepo/migrate-dbt-postgres branch January 24, 2025 18:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla:yes The PR author has signed the CLA
Projects
None yet
Development

Successfully merging this pull request may close these issues.