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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to ourterms of serviceand privacy statement.We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fabric data source #2034

Open
wants to merge 22 commits into
base: main
Choose a base branch
from
Open

Fabric data source #2034

wants to merge 22 commits into from

Conversation

JasperHDB
Copy link

This PR is an adaption to include a data source for Microsoft Fabric.

The adaption makes use of theazure-identitylibrary to authenticate to Azure Active Directory as aservice tenant.

Exampleconfiguration.ymlfile:

data_source warehouse:
type:fabric
connection:
#Connection
driver:'ODBC Driver 18 for SQL Server'#optional, default: ODBC Driver 18 for SQL Server
host:'jxz...rhy.datawarehouse.fabric.microsoft'
port:'1433'#optional, default: 1433
database:'raw'
schema:'dbo'
trusted_connection:false#optional, default: false
encrypt:false#optional, default: false
trust_server_certificate:false#optional, default: false

#Credentials
tenant_id:'<tenant_id>'
tenant_client_id:'<tenant_client_id>'
tenant_client_secret:'<tenant_client_secret>'

@benjamin-pirotte
Copy link

Hi, thank you for your contribution 🙏@m1n0will review it shortly and provide feedback if needed.

@JasperHDB
Copy link
Author

Hey, just wanted to check up on this PR. Let me know if I can help with anything to get this merged:)

@sonarcloud SonarCloud
Copy link

sonarcloud bot commented May 14, 2024

Quality Gate PassedQuality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Sep 9, 2024

Quality Gate FailedQuality Gate failed

Failed conditions
1 Security Hotspot

See analysis details on SonarCloud

@sdebruyn
Copy link
Contributor

#2160supersedes this one

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@JasperHDB @benjamin-pirotte @sdebruyn