Skip to content
This repository was archived by the owner on May 9, 2025. It is now read-only.

Conversation

ijac13
Copy link
Collaborator

@ijac13 ijac13 commented Apr 16, 2025

@ijac13 ijac13 requested a review from kentwelcome April 16, 2025 22:46
@ijac13 ijac13 requested review from wcchang1115 and removed request for kentwelcome April 17, 2025 05:39
@ijac13
Copy link
Collaborator Author

ijac13 commented Apr 17, 2025

@wcchang1115 update as we discussed. Will check with @doriwilson tomorrow.


Recce supports schema discovery across all major warehouses without needing to modify your `profiles.yml`.

However, if you're using DuckDB, you’ll need to explicitly define both schemas. For example:
Copy link
Contributor

Choose a reason for hiding this comment

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

Hi Karen,

However, if you use duckdb, you need to add a separate schemas in your dbt profile so you can run locally

I think we need to keep the original explanation -> "it's to make DuckDB able to run locally"

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

updated. I'll merge first in case I don't have time to do that next week. and we can update later

Copy link
Contributor

@wcchang1115 wcchang1115 left a comment

Choose a reason for hiding this comment

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

Except for the above comment, others are good to me.

@ijac13 ijac13 merged commit f04e3c1 into main Apr 18, 2025
@ijac13 ijac13 deleted the update-getting-started branch April 18, 2025 18:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants