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

riab -dq error #106

Closed
lore-edencehealth opened this issue Oct 10, 2024 · 3 comments · Fixed by #108
Closed

riab -dq error #106

lore-edencehealth opened this issue Oct 10, 2024 · 3 comments · Fixed by #108
Assignees
Labels
question Further information is requested

Comments

@lore-edencehealth
Copy link

sqlalchemy.exc.ProgrammingError: (pymssql.exceptions.ProgrammingError) (208, b"Invalid object name '[omop.dbo].[cdm_source'.DB-Lib error message 20018, severity 16:\nGeneral SQL Server error: Check messages from the SQL Server\n")
[SQL: select * from [[omop].[dbo]].[cdm_source];]

running select * from [[omop].[dbo]].[cdm_source]; gives an error,
replacing it by select * from [omop].[dbo].[cdm_source]; works so I think the extra brackets need to be removed from the code.

@pjlammertyn
Copy link
Contributor

Can you give more detail where the error occurs.

@pjlammertyn pjlammertyn self-assigned this Oct 11, 2024
@pjlammertyn pjlammertyn added the question Further information is requested label Oct 11, 2024
@pjlammertyn
Copy link
Contributor

More info on the square brackets: Delimited identifiers

@lore-edencehealth
Copy link
Author

https://github.com/RADar-AZDelta/Rabbit-in-a-Blender/blob/main/src/riab/etl/sql_server/data_quality.py#L79
I think the problem is that the brackets are around the combination of schema.table which makes it think the dot is part of the table name.

@pjlammertyn pjlammertyn linked a pull request Oct 15, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants