davsclaus opened a new pull request, #24715:
URL: https://github.com/apache/camel/pull/24715

   ## Summary
   
   - Move `quartzScheduler.scheduleJob()` / `rescheduleJob()` from `doStart()` 
to `startScheduler()` in `QuartzScheduledPollConsumerScheduler`, aligning with 
the `ScheduledPollConsumerScheduler` SPI contract
   - Same structural fix as CAMEL-24066 (Spring scheduler) — `doStart()` now 
only prepares resources, `startScheduler()` begins actual scheduling
   - Track scheduling state with a `scheduled` flag and reset it on unschedule 
for correct restart behavior
   - Add test verifying `startScheduler=false` prevents polling with 
`scheduler=quartz`
   
   ## Test plan
   
   - [x] New `FileConsumerQuartzSchedulerStartSchedulerTest` — verifies no 
messages received with `startScheduler=false`
   - [x] Existing `FileConsumerQuartzSchedulerTest` — basic quartz scheduler 
poll
   - [x] Existing `FileConsumerQuartzSchedulerRestartTest` — stop/start with 
existing trigger key
   - [x] Existing `FromFileQuartzSchedulerTest` — file consumer with quartz 
trigger parameters
   
   _Claude Code on behalf of davsclaus_
   
   Co-Authored-By: Claude Opus 4.6 <[email protected]>


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to