This repository was archived by the owner on May 2, 2023. It is now read-only.
Support connections to DuckDB for data-diff --dbt
#22
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
resolves #21
Overview
This PR probably isn't the long-term solution that you want. But it was a quick-n-easy way for me to get moving again for this demo.
Totally fine with me if you want to fix the problem below a different way!
Details
I was trying to run the following command when using a custom fork that added support for duckdb connections:
But I got the following error:
Since I wasn't super interested in discovering why this was happening or how best to solve it, this PR represents my quickest solution to the problem.
Here's the full stack trace: