Skip to content

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

@rgargour
Copy link

I’m currently working with Microsoft Fabric and wanted to check on the status of this PR. It would be great to use Elementary with Fabric. @999999333 🙏

@999999333
Copy link
Author

@rgargour It's up to elementary team. I did my part 🙃

@NoyaOffer
Copy link
Contributor

Hey! This contribution was an incredible starting point but it required some more work on our side, which is now in lower prioritization.
If @999999333 or anyone else here would want to work with us to finish this, we'll be happy to help with the contribution!

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.

6 participants