Skip to content

Conversation

@bradcrossen
Copy link
Contributor

@bradcrossen bradcrossen commented Jun 29, 2023

Support for SQLAlchemy 1.3 was removed in version 0.0.203 by change #6086. Re-adding support.

A similar change was merged before: #4647

Support for SQLAlchemy 1.3 was removed in version 0.0.203 by change langchain-ai#6086. Re-adding support.
@vercel
Copy link

vercel bot commented Jun 29, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Jun 29, 2023 9:23pm

@bradcrossen
Copy link
Contributor Author

Should be fixed!

@bradcrossen
Copy link
Contributor Author

@hwchase17 Friendly bump! Hoping to get this merged before the holiday weekend! 🙏

@rlancemartin rlancemartin self-assigned this Jun 29, 2023
@rlancemartin
Copy link
Contributor

@hwchase17 Friendly bump! Hoping to get this merged before the holiday weekend! 🙏

I will review now. Just kicked off tests.

@rlancemartin
Copy link
Contributor

@bradcrossen pls have a quick look and confirm you are OK w / this. it looks like the first import was not needed. i cleaned it up for you.

@bradcrossen
Copy link
Contributor Author

That is fine with me! Thanks!

@rlancemartin rlancemartin merged commit 9ca1cf0 into langchain-ai:master Jun 29, 2023
vowelparrot pushed a commit that referenced this pull request Jul 4, 2023
Support for SQLAlchemy 1.3 was removed in version 0.0.203 by change
#6086. Re-adding support.

- Description: Imports SQLAlchemy Row at class creation time instead of
at init to support SQLAlchemy <1.4. This is the only breaking change and
was introduced in version 0.0.203 #6086.
  
A similar change was merged before:
#4647
  
  - Dependencies: Reduces SQLAlchemy dependency to > 1.3
  - Tag maintainer: @rlancemartin, @eyurtsev, @hwchase17, @wangxuqi

---------

Co-authored-by: rlm <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants