Skip to content

Conversation

@Xnopyt
Copy link
Collaborator

@Xnopyt Xnopyt commented Aug 13, 2025

This adds batching support to the DbScheduler and SQS JobQueue, which will be used by the monitor command to improve thoughput.
There is now a batchable version of both SchedulerInterface and JobQueueInterface, which are used to determine if batching is supported.
DbScheduler has a configurable batch size to dermine how many records it will fetch. SQS will accept any number of jobs, but internally will split it into batches of 10 jobs per queue (which is the maximum AWS will accept at once).

@Xnopyt Xnopyt requested a review from chrisminett August 13, 2025 11:08
@codecov
Copy link

codecov bot commented Aug 13, 2025

Codecov Report

❌ Patch coverage is 70.00000% with 21 lines in your changes missing coverage. Please review.
✅ Project coverage is 65.43%. Comparing base (dcfaa53) to head (85f37a1).
⚠️ Report is 3 commits behind head on 2.x.x.

Files with missing lines Patch % Lines
src/Console/MonitorCommand.php 0.00% 18 Missing ⚠️
src/AwsSqs/JobQueue.php 90.00% 2 Missing ⚠️
src/Scheduler/DbScheduler.php 96.87% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##              2.x.x      #14      +/-   ##
============================================
- Coverage     65.50%   65.43%   -0.07%     
- Complexity      103      120      +17     
============================================
  Files            10       10              
  Lines           345      434      +89     
============================================
+ Hits            226      284      +58     
- Misses          119      150      +31     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Xnopyt Xnopyt force-pushed the dbscheduler-batching branch 2 times, most recently from 5dc0833 to b36361a Compare August 13, 2025 13:27
@Xnopyt Xnopyt force-pushed the dbscheduler-batching branch from b36361a to 5484baa Compare August 13, 2025 13:27
@Xnopyt Xnopyt force-pushed the dbscheduler-batching branch from 5484baa to 85f37a1 Compare August 13, 2025 13:30
@chrisminett chrisminett added the minor-version Release SHOULD be minor version increment label Aug 13, 2025
@Xnopyt Xnopyt merged commit 22ea0e9 into phlib:2.x.x Aug 15, 2025
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

minor-version Release SHOULD be minor version increment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants