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.

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

[BUG] When database (MS SQLServer) has case sensitive collation SQL_Latin1_General_CP1_CS_AS the Latest_open cte table is not found because incorrect capital letter #182

Closed
koillinengit opened this issue Feb 3, 2023 · 5 comments
Assignees
Labels
bug Something isn't working

Comments

@koillinengit
Copy link

koillinengit commented Feb 3, 2023

Describe the bug
When database (sqlserver) has case sensitive collation SQL_Latin1_General_CP1_CS_AS the Latest_open cte table is not found because incorrect capital letter

Environment

dbt version: 1.2.4
dbtvault version: 0.9.3
Database/Platform: Sqlserver

To Reproduce
Steps to reproduce the behavior:

  1. dbt run
  2. See error: ('42S02', "[42S02] [Microsoft][ODBC Driver 17 for SQL Server][SQL Server]Invalid object name 'Latest_open'. (208) (SQLExecDirectW)")

Expected behavior
Dbt run command for effectivity satellites should go without errors even if database has case sensitive collation: SQL_Latin1_General_CP1_CS_AS instead of case insensitive SQL_Latin1_General_CP1_CI_AS

Screenshots
image

Log files
('42S02', "[42S02] [Microsoft][ODBC Driver 17 for SQL Server][SQL Server]Invalid object name 'Latest_open'. (208) (SQLExecDirectW)")

Additional context
Add any other context about the problem here.

@koillinengit koillinengit added the bug Something isn't working label Feb 3, 2023
@DVAlexHiggs
Copy link
Member

DVAlexHiggs commented Feb 3, 2023

dbtvault does not currently support Azure Synapse, only the following are supported:

  • Snowflake
  • Bigquery
  • Postgres
  • Databricks
  • MS SQLServer

Please our documentation for more details.

Because we do not yet support Synapse, you will get unpredictable behavior such as this.

Until we add Azure Synapse support, you may extend dbtvault and add in Azure Synapse specific code yourself.

Please see the dbt docs for more info on this as well.

This is quite a simple fix from our side and if it's already working other than this, then we can add this in the next update. The capital 'L' is not intended so it should be changed. Thanks for this report.

@koillinengit
Copy link
Author

koillinengit commented Feb 3, 2023

The same issue is in MS SQLServer

@koillinengit koillinengit changed the title [BUG] When database (azuresql or synapse) has case sensitive collation SQL_Latin1_General_CP1_CS_AS the Latest_open cte table is not found because incorrect capital letter [BUG] When database (MS SQLServer) has case sensitive collation SQL_Latin1_General_CP1_CS_AS the Latest_open cte table is not found because incorrect capital letter Feb 3, 2023
@koillinengit
Copy link
Author

koillinengit commented Feb 3, 2023

In the snowflake version this seems to be fixed (which is case sensitive by default), but not in thisSQLserver version...
image

@DVAlexHiggs
Copy link
Member

In the snowflake version this seems to be fixed (which is case sensitive by default), but not in thisSQLserver version...
image

Thank you. We'll get this fixed in the next release. Must have slipped through QC!

@DVAlexHiggs
Copy link
Member

Hi! Fixed in 0.9.5. Thanks for your patience.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants