Skip to content

override new multi-db support in dbt-sqlserver#49

Merged
dataders merged 7 commits intomasterfrom
try-new-use-statement
Jun 2, 2021
Merged

override new multi-db support in dbt-sqlserver#49
dataders merged 7 commits intomasterfrom
try-new-use-statement

Conversation

@dataders
Copy link
Copy Markdown
Contributor

@dataders dataders commented May 8, 2021

override @semcha's new functionality in dbt-sqlserver (dbt-sqlserver #126) that allows for cross-database queries. Azure Synapse does not support this, so the sqlserver__ adapter macros that were previously used by dbt-synapse had to be re-implemented as synapse__ macros.

  • synapse__list_relations_without_caching
  • synapse__list_schemas
  • synapse__drop_relation

two other small changes:

  • synapse__drop_schema now just disptaches to dbt-sqlserver as it's logic was upstreamed into dbt-sqlserver
  • cleaned up redundant logic in synapse__drop_relation() and synapse__drop_relation_script()

@dataders dataders changed the title force install from semcha's PR override new multi-db support in dbt-sqlserver May 8, 2021
-rrequirements.txt
-e. No newline at end of file
-e.
git+https://github.com/semcha/dbt-sqlserver No newline at end of file
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this PR is pending until dbt-sqlserver==0.19.1 is released.

@dataders dataders marked this pull request as ready for review May 8, 2021 20:55
@dataders dataders merged commit 29bf8d8 into master Jun 2, 2021
@sdebruyn sdebruyn deleted the try-new-use-statement branch June 4, 2022 08:49
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