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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

DB name escape fix #94

Open

Oracen wants to merge 4 commits into dbt-msft:main from Oracen:main

Oracen commented Feb 28, 2023 •

edited

Loading

This PR is a minor fix to fix TSQL escape characters for some DB utility functions

Changes:

Added square brackets around DB names in drop_old_relations.sql and get_tables_by_pattern.sql
Updates get_tables_by_pattern to match the expected dbtutils format

Oracen added 4 commits

February 27, 2023 14:53


          Add escape for non-standard DB names

c60cf3d


          fix: Prevent local settings from hitting others' machines

23d3f95


          Add comments to explain square brackets

80f49ff


          Update get tables by pattern to match dbt_utils format

76848b8

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment