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

feat: fix Postgres primary keys growing #224

Merged
merged 6 commits into from
Sep 27, 2023
Merged

Conversation

wuzhican
Copy link
Contributor

@wuzhican wuzhican commented Sep 27, 2023

Fixed the issue of growing primary keys in postgres

Fix: #223

@casbin-bot
Copy link
Member

@tangyang9464 @JalinWang @imp2002 please review

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


wuzhican seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@hsluoyz
Copy link
Member

hsluoyz commented Sep 27, 2023

@wuzhican fix:

image

@hsluoyz
Copy link
Member

hsluoyz commented Sep 27, 2023

@wuzhican and use English commit messages

wuzhican added 2 commits September 27, 2023 22:13
fix the test error Error 1065: Query was empty
@wuzhican
Copy link
Contributor Author

Sorry, my golang is not very skilled. Please check it again

@hsluoyz hsluoyz changed the title fix: postgres primary keys growing feat: fix Postgres primary keys growing Sep 27, 2023
@hsluoyz hsluoyz merged commit b48be9f into casbin:master Sep 27, 2023
3 of 4 checks passed
@github-actions
Copy link

🎉 This PR is included in version 3.20.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

id increase when savePolicy
4 participants