search

meltano / sdk

Write 70% less code by using the SDK to build custom extractors and loaders that adhere to the Singer standard: https://sdk.meltano.com
https://sdk.meltano.com
Apache License 2.0
88 stars 64 forks source link

break: Require SQLAlchemy 2.0 for SQL taps and targets #1667

Open edgarrmondragon opened 1 year ago

edgarrmondragon commented 1 year ago

Feature scope

Taps (catalog, state, stream maps, etc.)

Description

We should require new taps and targets use SQLAlchemy 2.0 to avoid having to migrate them later, and come up with a migration plan for existing ones.

Most of the compatibility work was done in https://github.com/meltano/sdk/issues/1357, but individual connectors may still be using deprecated APIs.

Related:

edgarrmondragon commented 1 year ago

https://github.com/meltano/sdk/pull/1484/ adds support for SQLAlchemy 2.* without introducing breaking changes

edgarrmondragon commented 11 months ago

Closed by https://github.com/meltano/sdk/pull/1916

edgarrmondragon commented 2 weeks ago

This isn't in main yet, so I'll reopen it. It's tracked by https://github.com/meltano/sdk/milestone/2 anyway.

edgarrmondragon commented 2 weeks ago

https://github.com/snowflakedb/snowflake-sqlalchemy/pull/514 suggests that snowflake-sqlalchemy 1.6 will support SQLAlchemy 2.0.