Add postgresql database support #475
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
一个令人十分激动的 PR，我会尽快 review |
alembic直接迁移生成的迁移脚本在识别用with_variant实现数据库兼容的字段的时候会不准确，我一起提交了修正以后得migrate脚本文件，看看是否有必要，该文件我在mysql和pg都进行了测试 |
|
这会增加维护成本，我会调研一下这些配置带来的影响 |
# Conflicts: # backend/alembic/env.py # backend/app/admin/service/auth_service.py
|
alembic issue: sqlalchemy/alembic#1583 |
wu-clan
changed the title
wu-clan
approved these changes
Dec 25, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#353 做了PG的数据库支持，默认使用mysql，可以通过DB_TYPE指定数据库类型