ericmerrill / Banner-Luminis-Message-Broker-for-Moodle

A plugin for Moodle to import Banner/Luminis Message Broker data
13 stars 8 forks source link

Remove LMB cron from legacy crons #29

Open heather-williams opened 4 years ago

heather-williams commented 4 years ago

The enrol_lmb cron is currently running in the legacy cron tasks. This means any time the legacy cron is triggered it runs the task to process the extract folder. This cron can run a long time if you have large xml files (as some schools do) and it is possibly running the same files over and over again, even though they have not changed.

This task should be made in to a scheduled task that the user can control the timing of via the scheduled task UI. That way if those files only change once a day, that cron can be scheduled to run once a day. This will avoid the process eating resources over and over again to run files that haven't changed since the last run.

ericmerrill commented 4 years ago

Thanks for reporting this. I will see what I can do.

aweed1 commented 8 months ago

Just wanted to point out that support for legacy cron was recently removed by Moodle, which caused some confusion as the plugin still uses the legacy cron system. We got around this by running the cron task at an OS level.