Skip to content

Fix auto-increment id for postgres init data#632

Merged
wu-clan merged 4 commits into
fastapi-practices:masterfrom
huyuwei1996:master
May 28, 2025
Merged

Fix auto-increment id for postgres init data#632
wu-clan merged 4 commits into
fastapi-practices:masterfrom
huyuwei1996:master

Conversation

@huyuwei1996

Copy link
Copy Markdown
Contributor

#627

可以解决上述问题

@huyuwei1996

Copy link
Copy Markdown
Contributor Author

pg 我之前有测过,mysql 的麻烦确认下

@wu-clan

wu-clan commented May 26, 2025

Copy link
Copy Markdown
Member

或许这只是 pg 的问题,我在 mysql 中从未遇到过

@huyuwei1996 huyuwei1996 changed the title fix: Reset auto-increment values for all tables based on max id fix: Reset auto-increment id for postgres init data May 26, 2025
@huyuwei1996

Copy link
Copy Markdown
Contributor Author

已经删除mysql init sql的变更

@huyuwei1996 huyuwei1996 changed the title fix: Reset auto-increment id for postgres init data Fix auto-increment id for postgres init data May 27, 2025
@wu-clan

wu-clan commented May 27, 2025

Copy link
Copy Markdown
Member

Looks good.

Comment thread backend/sql/postgresql/init_test_data.sql Outdated
@wu-clan

wu-clan commented May 27, 2025

Copy link
Copy Markdown
Member

Otherwise LGTM.

@huyuwei1996

Copy link
Copy Markdown
Contributor Author

https://github.com/fastapi-practices/casbin_rbac 项目没有初始化 sql 的地方吗?本来打算一块改下

@wu-clan

wu-clan commented May 28, 2025

Copy link
Copy Markdown
Member

https://github.com/fastapi-practices/casbin_rbac 项目没有初始化 sql 的地方吗?本来打算一块改下

确实没加 sql 😂,默认启动自动创建表了

@wu-clan
wu-clan merged commit 5f3823c into fastapi-practices:master May 28, 2025
chuxijin pushed a commit to chuxijin/fba that referenced this pull request Feb 8, 2026
* fix: Reset auto-increment values for all tables based on max id

* refactor: Remove auto-increment reset for mysql

* remove unused code
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