Skip to content

✨ dbt-fabric adapter compatibility #777

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

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
Contributor

Exciting! Thanks for your contribution. @999999333

@nickteff
Copy link

nickteff commented Mar 6, 2025

@999999333 have you been able to integrate elementary with your changes into dbt-fabric project?

@NoyaOffer
Copy link
Contributor

Hey Karel @999999333 ! We're very excited about this. Can you reach out on Slack so we can schedule a call with our CTO?

@kdmstrahl
Copy link

Got this to work on a oldschool onprem SQL server as well! Thanx @999999333 !

@nickteff
Copy link

@kdmstrahl can you drop in the steps you took to do so? I would love a few pointers to get me going

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.

5 participants