Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • A aurweb
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 87
    • Issues 87
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 5
    • Merge requests 5
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Wiki
    • Wiki
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Arch LinuxArch Linux
  • aurweb
  • Merge requests
  • !547

fix: retry transactions who fail due to deadlocks

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Kevin Morris requested to merge kevr/aurweb:db-deadlock-retry into master Sep 13, 2022
  • Overview 0
  • Commits 1
  • Pipelines 2
  • Changes 16

In my opinion, this kind of handling of transactions is pretty ugly. The being said, we have issues with running into deadlocks on aur.al, so this commit works against that immediate bug.

An ideal solution would be to deal with retrying transactions through the db.begin() scope, so we wouldn't have to explicitly annotate functions as "retry functions," which is what this commit does.

Closes #376 (closed)

Signed-off-by: Kevin Morris kevr@0cost.org

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: db-deadlock-retry