-
Notifications
You must be signed in to change notification settings - Fork 418
Document sql source reflection level and type adapter #1467
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
Conversation
✅ Deploy Preview for dlt-hub-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
| import dlt | ||
| from snowflake.sqlalchemy import TIMESTAMP_NTZ | ||
| import sqlalchemy as sa | ||
| from sqlalchemy.sql.sqltypes import TypeAdapter |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tests fail with
Module "sqlalchemy.sql.sqltypes" has no attribute "TypeAdapter"
may be you meant from pydantic import TypeAdapter ? or what version of sqlalchemy you have?
AstrakhantsevaAA
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @steinitzu, could you fix the snippet, sqlalchemy.sql.sqltype doesn't have TypeAdapter
758dd48 to
11d0cbb
Compare
11d0cbb to
56f454e
Compare
Document sql source reflection level and type adapter (#1467)
Description
Docs for dlt-hub/verified-sources#487
Related Issues
Additional Context