snowflakedb / snowflake-sqlalchemy

Snowflake SQLAlchemy
https://pypi.python.org/pypi/snowflake-sqlalchemy/
Apache License 2.0
235 stars 152 forks source link

SNOW-930985: Add support for SQLAlchemy 2 #451

Closed zmievsa closed 1 year ago

zmievsa commented 1 year ago

What is the current behavior?

You can't install snowflake-sqlalchemy with the second version of sqlalchemy

What is the desired behavior?

To install snowflake-sqlalchemy with the second version of sqlalchemy

How would this improve snowflake-connector-python?

It would allow you to use all improvements from sqlalchemy 2. To me, the most important ones are the type hinting improvements but there is a lot more.

sfc-gh-achandrasekaran commented 1 year ago

@Ovsyanka83 we are aware of this request. It is in our backlog and will be prioritized in the next couple of quarters

sk- commented 1 year ago

@sfc-gh-achandrasekaran why was this issue closed if it's not yet resolved?

Also, could you share a better estimate. Do you mean in the next couple of quarters you will start discussing the priority or that in the next couple of quarters you will deliver. Either way taking so much time to bring your package up to date with their dependencies is quite unacceptable.

sfc-gh-achandrasekaran commented 1 year ago

@sk- apologies. I closed it because we are already tracking this using https://github.com/snowflakedb/snowflake-sqlalchemy/issues/452

We will start working on this in the next couple of quarters. We totally get the priority of this ask but we also need to balance this with other competing customer asks/priorities.