Skip to content

Update StatisticsMaintenanceCronjob.php - #149

Merged
skerbis merged 1 commit into
mainfrom
speedup2
Aug 4, 2026
Merged

Update StatisticsMaintenanceCronjob.php#149
skerbis merged 1 commit into
mainfrom
speedup2

Conversation

@skerbis

@skerbis skerbis commented Aug 4, 2026

Copy link
Copy Markdown
Member

No description provided.

Copilot AI lite review requested due to automatic review settings August 4, 2026 13:24
@skerbis
skerbis merged commit b7b3d24 into main Aug 4, 2026
1 check passed

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the statistics maintenance cronjob to be more resilient to transient database lock conflicts during cleanup operations, reducing the chance that the whole maintenance run fails due to lock wait timeouts or deadlocks.

Changes:

  • Add tracking/reporting of deferred delete batches when deletes are skipped due to persistent DB locks.
  • Increase delete retry attempts and introduce a small linear backoff between retries.
  • Centralize lock-timeout/deadlock detection into a helper (isRetryableLockException()), including support for error codes 1205 and 1213.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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