dbt-labs / dbt-labs-experimental-features

dbt support for database features which are not yet supported natively in dbt-core
Apache License 2.0
145 stars 43 forks source link

insert_by_period_materialization.sql is doulbe quoting identifiers #33

Open jeff-skoldberg opened 2 years ago

jeff-skoldberg commented 2 years ago

Describe the error

The macro is double quoting identifiers even though my config is not set to. First error: "MY_DB"."my_schema" does not exist or not authorized. I went to settings, Account Settings, Credentials, and changed the schema to upper case. Next Error "MY_DB."MY_SCHEMA"."test" does not exist I changed the model name to upper case and it worked.

Steps to reproduce

create a model with a lowercase name in a database that has default uppercase or vice versa.

Expected results

case insensitivity

Actual results

Screenshots and log output

System information

The contents of your packages.yml file: packages:

Which database are you using dbt with?

The output of dbt --version:

None

Additional context

Are you interested in contributing the fix?

joellabes commented 2 years ago

@jeff-skoldberg this macro is not currently supported for Snowflake - I'm not surprised that this sort of thing has come up!

Also, it has just moved to https://github.com/dbt-labs/dbt-labs-experimental-features/tree/main/insert_by_period (it will be removed from dbt utils in v1.0.0 later next month).

I'm going to transfer this issue there, but it's not a project that the dbt team is dedicating any time to so it'll be up to the wider community to implement Snowflake compatibility.