Skip to content

feat: add Microsoft Fabric Warehouse / Azure SQL support via Service Principal#838

Open
Germain-D wants to merge 2 commits into
k1LoW:mainfrom
Germain-D:main
Open

feat: add Microsoft Fabric Warehouse / Azure SQL support via Service Principal#838
Germain-D wants to merge 2 commits into
k1LoW:mainfrom
Germain-D:main

Conversation

@Germain-D

Copy link
Copy Markdown
  • New azuresql:// datasource — connects to Microsoft Fabric Warehouse and Azure SQL using Azure AD
    Service Principal auth (fedauth=ActiveDirectoryServicePrincipal). Delegates schema analysis to the
    existing mssql driver.

    • Replace FOR XML PATH with STRING_AGG — FOR XML PATH is unsupported in Synapse SQL / Microsoft Fabric
      Warehouse. STRING_AGG is supported on Azure SQL, Fabric Warehouse, and SQL Server 2017+.

    Breaking Change

    SQL Server 2016 and earlier no longer supported (no STRING_AGG).

    Files changed

    • datasource/azuresql.go — new datasource + STRING_AGG fix
    • datasource/azuresql_test.go — tests for both
    • datasource/datasource.go — register new datasource
    • drivers/mssql/mssql.go — remove FOR XML PATH
    • go.mod / go.sum — new Azure SDK dependency

Germain-D added 2 commits June 4, 2026 11:34
  Enables connecting to Microsoft Fabric Warehouse via azuresql://
  using fedauth=ActiveDirectoryServicePrincipal. Bypasses dburl
  and delegates analysis to the existing mssql driver.
  FOR XML PATH is unsupported in Synapse SQL / Microsoft Fabric Warehouse.
  STRING_AGG achieves the same concatenation and is supported across
  Azure SQL, Fabric Warehouse, and SQL Server 2017+.

  BREAKING CHANGE: SQL Server 2016 and earlier no longer supported.
@Germain-D Germain-D changed the title Add Microsoft Fabric Warehouse / Azure SQL support via Service Principal feat: add Microsoft Fabric Warehouse / Azure SQL support via Service Principal Jun 6, 2026
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.

1 participant