Closed mikekutzma closed 7 months ago
feat: Make sqlalchemy_extractor compatible with sqlalchemy>=1.4 (Moved from #2209)
These changes to the sql_alchemy_extractor module allow for using the extractors with updated versions of sqlalchemy.
sql_alchemy_extractor
sqlalchemy
This addresses issues such as
text
SQLAlchemy>=2.0
Row
SQLAlchemy>=1.4
Tested using sampple_mssql_metadata.py script for
sampple_mssql_metadata.py
SQLAlchemy==1.3.16
SQLAlchemy==2.0.23
None
@ozandogrultan Anything else needed for this one? Happy to update if needed
Bumping for review
Awesome work, congrats on your first merged pull request!
feat: Make sqlalchemy_extractor compatible with sqlalchemy>=1.4 (Moved from #2209)
Description
These changes to the
sql_alchemy_extractor
module allow for using the extractors with updated versions ofsqlalchemy
.Motivation and Context
This addresses issues such as
2108 which require the
text
function to be used to wrap queries inSQLAlchemy>=2.0
Row
objects to dicts forSQLAlchemy>=1.4
How Has This Been Tested?
Tested using
sampple_mssql_metadata.py
script forSQLAlchemy==1.3.16
SQLAlchemy==2.0.23
Documentation
None
CheckList