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

✨ dbt-fabric adapter compatibility #777

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

999999333
Copy link

References

elementary-data/elementary#1742

Worth Mentioning - T-SQL nuances

  • T-SQL does not support boolean data types. As a workaround, this implementation uses the bit data type to reimplement boolean functionality.
  • Generic STDDEV macro. - stDev vs. stDDev
  • DATE/TIME formatting.
  • Explicit GROUP BY clauses.

Our Environment

We are currently running it in a production environment with ~200 models and tests for >1 month
dbt version: 1.9.1
Fabric version: 1.8.7
We use this setup to write dbt_* tables into the data warehouse.
A custom PowerBI report is built on top of these tables.
Note: We do not use the Elementary CLI, so this functionality has not been tested in this setup.

Limitations/Bugs

  • Incorrect row_inserted Count - This is a known limitation of the dbt-fabric connector.

  • Temporary Views Not Being Deleted - Observed that the initial temporary views are not getting deleted. Root cause is unknown.

Integration Tests

Unfortunately, I was unable to set up a testing environment for integration tests.
Would love some support on this one... 😊

@999999333 999999333 changed the title ✨🅵 dbt-fabric 🅵 adapter compatibility ✨ dbt-fabric adapter compatibility Jan 26, 2025
@MikaKerman
Copy link

Exciting! Thanks for your contribution. @999999333

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.

2 participants