Skip to content
This repository was archived by the owner on Dec 8, 2023. It is now read-only.

Sql alchemy v2#33

Merged
morgante merged 4 commits intogetgrit:mainfrom
SarthakSingh31:sql_alchemy_v2
Oct 2, 2023
Merged

Sql alchemy v2#33
morgante merged 4 commits intogetgrit:mainfrom
SarthakSingh31:sql_alchemy_v2

Conversation

@SarthakSingh31
Copy link
Contributor

@SarthakSingh31 SarthakSingh31 commented Sep 29, 2023

This is working on some migrations but not working on all.
The only one I was able to make it work with was https://github.com/blockchain-etl/ethereum-etl/tree/develop.

There are a lot of complications when migrating sqlalchemy.

  1. They might also be using flask-sqlalchemy which also needs to be upgraded to version >= 3.0.2.
  2. They also be using alembic which also needs to be upgraded usually.
  3. Some part of the migration sometimes starts raising errors within sqlalchemy itself. I have not been able to diagnose the source. (https://github.com/wazuh/wazuh is one of the repos that run into this issue)

@morgante morgante merged commit db46426 into getgrit:main Oct 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants