Skip to content

RM-10: Replace close request queue with precise scheduler - #3

Merged
BenHall-1 merged 1 commit into
masterfrom
RM-10
Aug 27, 2026
Merged

RM-10: Replace close request queue with precise scheduler#3
BenHall-1 merged 1 commit into
masterfrom
RM-10

Conversation

@biast12

@biast12 biast12 commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

RM-10

Description

Replace the queue-based close request system with a timer-based scheduler that fires requests at their exact close_at time. This eliminates the sweep interval delay and improves precision. The scheduler uses a reconciliation pattern to arm, re-arm, and cancel timers based on the current database state, handling cases where requests are denied, closed, excluded, or rescheduled. Also added a default sweep time of 1 second to the config.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Improvement

Testing

Open a few tickets and send a close request for each of them, with some space in between them.

Checklist

  • My code follows the style of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works

Replace the queue-based close request system with a timer-based scheduler that fires requests at their exact close_at time. This eliminates the sweep interval delay and improves precision. The scheduler uses a reconciliation pattern to arm, re-arm, and cancel timers based on the current database state, handling cases where requests are denied, closed, excluded, or rescheduled. Also added a default sweep time of 1 second to the config.
@biast12
biast12 requested a review from a team as a code owner July 2, 2026 22:02
@BenHall-1
BenHall-1 merged commit 69c043f into master Aug 27, 2026
2 checks passed
@BenHall-1
BenHall-1 deleted the RM-10 branch August 27, 2026 09:55
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