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.

Sign up for GitHub

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

Jump to bottom

Making locking parameters more intuitive #6719

Merged
merged 2 commits into from
Dec 15, 2023
Merged

Making locking parameters more intuitive #6719

merged 2 commits into from
Dec 15, 2023

Conversation

dogenkigen
Copy link
Contributor

@dogenkigen dogenkigen commented Dec 4, 2023 •
edited
Loading

  • Do only one thing
  • Non breaking API changes
  • Tested

What did this pull request do?

This PR provides constants for locking strength and locking option to make it easier to "guess". Partially solves #6720

User Case Description

db.Clauses(clause.Locking{
  Strength: clause.LockingStrengthUpdate,
  Options: clause.LockingOptionsNoWait,
}).Find(&users)

@dogenkigen dogenkigen mentioned this pull request Dec 4, 2023
Open
@jinzhu jinzhu merged commit b9ebdb1 into go-gorm:master Dec 15, 2023
35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@a631807682 a631807682 a631807682 approved these changes

@jinzhu jinzhu jinzhu approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dogenkigen @jinzhu @a631807682