Segfault-Inc / Multicorn

Data Access Library
https://multicorn.org/
PostgreSQL License
700 stars 145 forks source link

Update sqlalchemyfdw.py #213

Closed mshahbazi closed 4 years ago

mshahbazi commented 6 years ago

make it compatible with the recent sqlalchemy

galuszkak commented 5 years ago

@mshahbazi maybe it makes more sense to have support for old and new SQLAlchemy? I think we should issue DeprecationWarning, but not force users to upgrade SQLAlchemy immediately.

rdunklau commented 4 years ago

I merged this fix, wrapped in an "except ImportErrror" block to fall back to using the old names. Thanks.