Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dont error on sql import #4647

Merged
merged 1 commit into from
May 18, 2023
Merged

dont error on sql import #4647

merged 1 commit into from
May 18, 2023

Conversation

hwchase17
Copy link
Contributor

this makes it so we dont throw errors when importing langchain when sqlalchemy==1.3.1

we dont really want to support 1.3.1 (seems like unneccessary maintance cost) BUT we would like it to not terribly error should someone decide to run on it

@s16h
Copy link
Contributor

s16h commented May 17, 2023

@hwchase17, bumping this for a cheeky merge and release :)

@hwchase17
Copy link
Contributor Author

sorry @s16h , will be in next release!

@hwchase17 hwchase17 merged commit d5a0704 into master May 18, 2023
@hwchase17 hwchase17 deleted the harrison/sqlalchemy-backwards branch May 18, 2023 16:27
@danielchalef danielchalef mentioned this pull request Jun 5, 2023
This was referenced Jun 25, 2023
rlancemartin added a commit that referenced this pull request Jun 29, 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 <pexpresss31@gmail.com>
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 <pexpresss31@gmail.com>
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.

2 participants