From 4bb70d759342778538b1f51f7ff43c10aa07a233 Mon Sep 17 00:00:00 2001 From: Michael Date: Tue, 1 Sep 2020 08:09:16 +0000 Subject: [PATCH 1/6] Split cron tasks in several worker tasks --- src/Worker/CheckdeletedContacts.php | 41 +++++++ src/Worker/ClearCache.php | 9 -- src/Worker/Cron.php | 168 +--------------------------- src/Worker/OptimizeTables.php | 57 ++++++++++ src/Worker/PollContacts.php | 132 ++++++++++++++++++++++ src/Worker/UpdatePublicContacts.php | 56 ++++++++++ 6 files changed, 290 insertions(+), 173 deletions(-) create mode 100644 src/Worker/CheckdeletedContacts.php create mode 100644 src/Worker/OptimizeTables.php create mode 100644 src/Worker/PollContacts.php create mode 100644 src/Worker/UpdatePublicContacts.php diff --git a/src/Worker/CheckdeletedContacts.php b/src/Worker/CheckdeletedContacts.php new file mode 100644 index 000000000..c47495211 --- /dev/null +++ b/src/Worker/CheckdeletedContacts.php @@ -0,0 +1,41 @@ +. + * + */ + +namespace Friendica\Worker; + +use Friendica\Core\Worker; +use Friendica\Database\DBA; + +/** + * Checks for contacts that are about to be deleted and ensures that they are removed. + * This should be done automatically in the "remove" function. This here is a cleanup job. + */ +class CheckdeletedContacts +{ + public static function execute() + { + $contacts = DBA::select('contact', ['id'], ['deleted' => true]); + while ($contact = DBA::fetch($contacts)) { + Worker::add(PRIORITY_MEDIUM, 'RemoveContact', $contact['id']); + } + DBA::close($contacts); + } +} diff --git a/src/Worker/ClearCache.php b/src/Worker/ClearCache.php index a1e78434c..5eee4c74a 100644 --- a/src/Worker/ClearCache.php +++ b/src/Worker/ClearCache.php @@ -37,9 +37,6 @@ class ClearCache // clear old cache DI::cache()->clear(); - if (DI::config()->get('system', 'optimize_tables')) { - DBA::e("OPTIMIZE TABLE `cache`"); - } // clear old item cache files clear_cache(); @@ -66,14 +63,8 @@ class ClearCache // Delete the cached OEmbed entries that are older than three month DBA::delete('oembed', ["`created` < NOW() - INTERVAL 3 MONTH"]); - if (DI::config()->get('system', 'optimize_tables')) { - DBA::e("OPTIMIZE TABLE `oembed`"); - } // Delete the cached "parse_url" entries that are older than three month DBA::delete('parsed_url', ["`created` < NOW() - INTERVAL 3 MONTH"]); - if (DI::config()->get('system', 'optimize_tables')) { - DBA::e("OPTIMIZE TABLE `parsed_url`"); - } } } diff --git a/src/Worker/Cron.php b/src/Worker/Cron.php index 1b4d5f8cd..ad50707c4 100644 --- a/src/Worker/Cron.php +++ b/src/Worker/Cron.php @@ -21,14 +21,11 @@ namespace Friendica\Worker; -use Friendica\Core\Addon; use Friendica\Core\Hook; use Friendica\Core\Logger; -use Friendica\Core\Protocol; use Friendica\Core\Worker; use Friendica\Database\DBA; use Friendica\DI; -use Friendica\Model\Contact; use Friendica\Util\DateTimeFormat; class Cron @@ -93,10 +90,10 @@ class Cron // check upstream version? Worker::add(PRIORITY_LOW, 'CheckVersion'); - self::checkdeletedContacts(); + Worker::add(PRIORITY_LOW, 'CheckdeletedContacts'); if (DI::config()->get('system', 'optimize_tables')) { - self::optimizeTables(); + Worker::add(PRIORITY_LOW, 'OptimizeTables'); } DI::config()->set('system', 'last_expire_day', $d2); @@ -140,10 +137,10 @@ class Cron } // Poll contacts - self::pollContacts(); + Worker::add(PRIORITY_HIGH, 'PollContacts'); // Update contact information - self::updatePublicContacts(); + Worker::add(PRIORITY_LOW, 'UpdatePublicContacts'); Logger::log('cron: end'); @@ -151,161 +148,4 @@ class Cron return; } - - /** - * Optimize tables that are known to grow and shrink all the time - * - * @return void - */ - private static function optimizeTables() - { - Logger::info('Optimize start'); - - DBA::e("OPTIMIZE TABLE `auth_codes`"); - DBA::e("OPTIMIZE TABLE `challenge`"); - DBA::e("OPTIMIZE TABLE `locks`"); - DBA::e("OPTIMIZE TABLE `profile_check`"); - DBA::e("OPTIMIZE TABLE `session`"); - DBA::e("OPTIMIZE TABLE `tokens`"); - - DI::lock()->release('optimize_tables'); - } - - /** - * Checks for contacts that are about to be deleted and ensures that they are removed. - * This should be done automatically in the "remove" function. This here is a cleanup job. - */ - private static function checkdeletedContacts() - { - $contacts = DBA::select('contact', ['id'], ['deleted' => true]); - while ($contact = DBA::fetch($contacts)) { - Worker::add(PRIORITY_MEDIUM, 'RemoveContact', $contact['id']); - } - DBA::close($contacts); - } - - /** - * Update public contacts - * - * @throws \Friendica\Network\HTTPException\InternalServerErrorException - */ - private static function updatePublicContacts() { - $count = 0; - $last_updated = DateTimeFormat::utc('now - 1 week'); - $condition = ["`network` IN (?, ?, ?, ?) AND `uid` = ? AND NOT `self` AND `last-update` < ?", - Protocol::ACTIVITYPUB, Protocol::DFRN, Protocol::DIASPORA, Protocol::OSTATUS, 0, $last_updated]; - - $oldest_date = ''; - $oldest_id = ''; - $contacts = DBA::select('contact', ['id', 'last-update'], $condition, ['limit' => 100, 'order' => ['last-update']]); - while ($contact = DBA::fetch($contacts)) { - if (empty($oldest_id)) { - $oldest_id = $contact['id']; - $oldest_date = $contact['last-update']; - } - Worker::add(PRIORITY_LOW, "UpdateContact", $contact['id']); - ++$count; - } - Logger::info('Initiated update for public contacts', ['interval' => $count, 'id' => $oldest_id, 'oldest' => $oldest_date]); - DBA::close($contacts); - } - - /** - * Poll contacts for unreceived messages - * - * @throws \Friendica\Network\HTTPException\InternalServerErrorException - */ - private static function pollContacts() { - Addon::reload(); - - $sql = "SELECT `contact`.`id`, `contact`.`nick`, `contact`.`name`, `contact`.`network`, `contact`.`archive`, - `contact`.`last-update`, `contact`.`priority`, `contact`.`rating`, `contact`.`rel`, `contact`.`subhub` - FROM `user` - STRAIGHT_JOIN `contact` - ON `contact`.`uid` = `user`.`uid` AND `contact`.`poll` != '' - AND `contact`.`network` IN (?, ?, ?, ?, ?) - AND NOT `contact`.`self` AND NOT `contact`.`blocked` - AND `contact`.`rel` != ? - WHERE NOT `user`.`account_expired` AND NOT `user`.`account_removed`"; - - $parameters = [Protocol::DFRN, Protocol::ACTIVITYPUB, Protocol::OSTATUS, Protocol::FEED, Protocol::MAIL, Contact::FOLLOWER]; - - // Only poll from those with suitable relationships, - // and which have a polling address and ignore Diaspora since - // we are unable to match those posts with a Diaspora GUID and prevent duplicates. - $abandon_days = intval(DI::config()->get('system', 'account_abandon_days')); - if ($abandon_days < 1) { - $abandon_days = 0; - } - - if (!empty($abandon_days)) { - $sql .= " AND `user`.`login_date` > UTC_TIMESTAMP() - INTERVAL ? DAY"; - $parameters[] = $abandon_days; - } - - $contacts = DBA::p($sql, $parameters); - - if (!DBA::isResult($contacts)) { - return; - } - - while ($contact = DBA::fetch($contacts)) { - $ratings = [0, 3, 7, 8, 9, 10]; - if (DI::config()->get('system', 'adjust_poll_frequency') && ($contact['network'] == Protocol::FEED)) { - $rating = $contact['rating']; - } elseif (array_key_exists($contact['priority'], $ratings)) { - $rating = $ratings[$contact['priority']]; - } else { - $rating = -1; - } - - // Friendica and OStatus are checked once a day - if (in_array($contact['network'], [Protocol::DFRN, Protocol::OSTATUS])) { - $rating = 8; - } - - // ActivityPub is checked once a week - if ($contact['network'] == Protocol::ACTIVITYPUB) { - $rating = 9; - } - - // Check archived contacts once a month - if ($contact['archive']) { - $rating = 10; - } - - if ($rating < 0) { - continue; - } - /* - * Based on $contact['priority'], should we poll this site now? Or later? - */ - - $min_poll_interval = DI::config()->get('system', 'min_poll_interval'); - - $poll_intervals = [$min_poll_interval . ' minute', '15 minute', '30 minute', - '1 hour', '2 hour', '3 hour', '6 hour', '12 hour' ,'1 day', '1 week', '1 month']; - - $now = DateTimeFormat::utcNow(); - $next_update = DateTimeFormat::utc($contact['last-update'] . ' + ' . $poll_intervals[$rating]); - - if (empty($poll_intervals[$rating]) || ($now < $next_update)) { - Logger::debug('No update', ['cid' => $contact['id'], 'rating' => $rating, 'next' => $next_update, 'now' => $now]); - continue; - } - - if ((($contact['network'] == Protocol::FEED) && ($contact['priority'] <= 3)) || ($contact['network'] == Protocol::MAIL)) { - $priority = PRIORITY_MEDIUM; - } elseif ($contact['archive']) { - $priority = PRIORITY_NEGLIGIBLE; - } else { - $priority = PRIORITY_LOW; - } - - Logger::log("Polling " . $contact["network"] . " " . $contact["id"] . " " . $contact['priority'] . " " . $contact["nick"] . " " . $contact["name"]); - - Worker::add(['priority' => $priority, 'dont_fork' => true, 'force_priority' => true], 'OnePoll', (int)$contact['id']); - } - DBA::close($contacts); - } } diff --git a/src/Worker/OptimizeTables.php b/src/Worker/OptimizeTables.php new file mode 100644 index 000000000..0f49a3616 --- /dev/null +++ b/src/Worker/OptimizeTables.php @@ -0,0 +1,57 @@ +. + * + */ + +namespace Friendica\Worker; + +use Friendica\Core\Logger; +use Friendica\Database\DBA; +use Friendica\DI; + +/** + * Optimize tables that are known to grow and shrink all the time + */ +class OptimizeTables +{ + public static function execute() + { + + if (!DI::lock()->acquire('optimize_tables', 0)) { + Logger::warning('Lock could not be acquired'); + return; + } + + Logger::info('Optimize start'); + + DBA::e("OPTIMIZE TABLE `auth_codes`"); + DBA::e("OPTIMIZE TABLE `cache`"); + DBA::e("OPTIMIZE TABLE `challenge`"); + DBA::e("OPTIMIZE TABLE `locks`"); + DBA::e("OPTIMIZE TABLE `oembed`"); + DBA::e("OPTIMIZE TABLE `parsed_url`"); + DBA::e("OPTIMIZE TABLE `profile_check`"); + DBA::e("OPTIMIZE TABLE `session`"); + DBA::e("OPTIMIZE TABLE `tokens`"); + + Logger::info('Optimize end'); + + DI::lock()->release('optimize_tables'); + } +} diff --git a/src/Worker/PollContacts.php b/src/Worker/PollContacts.php new file mode 100644 index 000000000..e73801f2b --- /dev/null +++ b/src/Worker/PollContacts.php @@ -0,0 +1,132 @@ +. + * + */ + +namespace Friendica\Worker; + +use Friendica\Core\Addon; +use Friendica\Core\Logger; +use Friendica\Core\Protocol; +use Friendica\Core\Worker; +use Friendica\Database\DBA; +use Friendica\DI; +use Friendica\Model\Contact; +use Friendica\Util\DateTimeFormat; + +/** + * Poll contacts for unreceived messages + */ +class PollContacts +{ + public static function execute() + { + Addon::reload(); + + $sql = "SELECT `contact`.`id`, `contact`.`nick`, `contact`.`name`, `contact`.`network`, `contact`.`archive`, + `contact`.`last-update`, `contact`.`priority`, `contact`.`rating`, `contact`.`rel`, `contact`.`subhub` + FROM `user` + STRAIGHT_JOIN `contact` + ON `contact`.`uid` = `user`.`uid` AND `contact`.`poll` != '' + AND `contact`.`network` IN (?, ?, ?, ?, ?) + AND NOT `contact`.`self` AND NOT `contact`.`blocked` + AND `contact`.`rel` != ? + WHERE NOT `user`.`account_expired` AND NOT `user`.`account_removed`"; + + $parameters = [Protocol::DFRN, Protocol::ACTIVITYPUB, Protocol::OSTATUS, Protocol::FEED, Protocol::MAIL, Contact::FOLLOWER]; + + // Only poll from those with suitable relationships, + // and which have a polling address and ignore Diaspora since + // we are unable to match those posts with a Diaspora GUID and prevent duplicates. + $abandon_days = intval(DI::config()->get('system', 'account_abandon_days')); + if ($abandon_days < 1) { + $abandon_days = 0; + } + + if (!empty($abandon_days)) { + $sql .= " AND `user`.`login_date` > UTC_TIMESTAMP() - INTERVAL ? DAY"; + $parameters[] = $abandon_days; + } + + $contacts = DBA::p($sql, $parameters); + + if (!DBA::isResult($contacts)) { + return; + } + + while ($contact = DBA::fetch($contacts)) { + $ratings = [0, 3, 7, 8, 9, 10]; + if (DI::config()->get('system', 'adjust_poll_frequency') && ($contact['network'] == Protocol::FEED)) { + $rating = $contact['rating']; + } elseif (array_key_exists($contact['priority'], $ratings)) { + $rating = $ratings[$contact['priority']]; + } else { + $rating = -1; + } + + // Friendica and OStatus are checked once a day + if (in_array($contact['network'], [Protocol::DFRN, Protocol::OSTATUS])) { + $rating = 8; + } + + // ActivityPub is checked once a week + if ($contact['network'] == Protocol::ACTIVITYPUB) { + $rating = 9; + } + + // Check archived contacts once a month + if ($contact['archive']) { + $rating = 10; + } + + if ($rating < 0) { + continue; + } + /* + * Based on $contact['priority'], should we poll this site now? Or later? + */ + + $min_poll_interval = DI::config()->get('system', 'min_poll_interval'); + + $poll_intervals = [$min_poll_interval . ' minute', '15 minute', '30 minute', + '1 hour', '2 hour', '3 hour', '6 hour', '12 hour' ,'1 day', '1 week', '1 month']; + + $now = DateTimeFormat::utcNow(); + $next_update = DateTimeFormat::utc($contact['last-update'] . ' + ' . $poll_intervals[$rating]); + + if (empty($poll_intervals[$rating]) || ($now < $next_update)) { + Logger::debug('No update', ['cid' => $contact['id'], 'rating' => $rating, 'next' => $next_update, 'now' => $now]); + continue; + } + + if ((($contact['network'] == Protocol::FEED) && ($contact['priority'] <= 3)) || ($contact['network'] == Protocol::MAIL)) { + $priority = PRIORITY_MEDIUM; + } elseif ($contact['archive']) { + $priority = PRIORITY_NEGLIGIBLE; + } else { + $priority = PRIORITY_LOW; + } + + Logger::notice("Polling " . $contact["network"] . " " . $contact["id"] . " " . $contact['priority'] . " " . $contact["nick"] . " " . $contact["name"]); + + Worker::add(['priority' => $priority, 'dont_fork' => true, 'force_priority' => true], 'OnePoll', (int)$contact['id']); + } + DBA::close($contacts); + } +} diff --git a/src/Worker/UpdatePublicContacts.php b/src/Worker/UpdatePublicContacts.php new file mode 100644 index 000000000..453606bfa --- /dev/null +++ b/src/Worker/UpdatePublicContacts.php @@ -0,0 +1,56 @@ +. + * + */ + +namespace Friendica\Worker; + +use Friendica\Core\Logger; +use Friendica\Core\Protocol; +use Friendica\Core\Worker; +use Friendica\Database\DBA; +use Friendica\Util\DateTimeFormat; + +/** + * Update public contacts + */ +class UpdatePublicContacts +{ + public static function execute() + { + $count = 0; + $last_updated = DateTimeFormat::utc('now - 1 week'); + $condition = ["`network` IN (?, ?, ?, ?) AND `uid` = ? AND NOT `self` AND `last-update` < ?", + Protocol::ACTIVITYPUB, Protocol::DFRN, Protocol::DIASPORA, Protocol::OSTATUS, 0, $last_updated]; + + $oldest_date = ''; + $oldest_id = ''; + $contacts = DBA::select('contact', ['id', 'last-update'], $condition, ['limit' => 100, 'order' => ['last-update']]); + while ($contact = DBA::fetch($contacts)) { + if (empty($oldest_id)) { + $oldest_id = $contact['id']; + $oldest_date = $contact['last-update']; + } + Worker::add(PRIORITY_LOW, "UpdateContact", $contact['id']); + ++$count; + } + Logger::info('Initiated update for public contacts', ['interval' => $count, 'id' => $oldest_id, 'oldest' => $oldest_date]); + DBA::close($contacts); + } +} From 34cce68dc623ede54539bde529f90f13515b677b Mon Sep 17 00:00:00 2001 From: Michael Date: Tue, 1 Sep 2020 08:11:42 +0000 Subject: [PATCH 2/6] Replaced deprecated logger calls --- src/Worker/Cron.php | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/Worker/Cron.php b/src/Worker/Cron.php index ad50707c4..3eb93ba26 100644 --- a/src/Worker/Cron.php +++ b/src/Worker/Cron.php @@ -41,12 +41,12 @@ class Cron if ($last) { $next = $last + ($poll_interval * 60); if ($next > time()) { - Logger::log('cron intervall not reached'); + Logger::notice('cron intervall not reached'); return; } } - Logger::log('cron: start'); + Logger::notice('cron: start'); // Fork the cron jobs in separate parts to avoid problems when one of them is crashing Hook::fork($a->queue['priority'], "cron"); @@ -142,7 +142,7 @@ class Cron // Update contact information Worker::add(PRIORITY_LOW, 'UpdatePublicContacts'); - Logger::log('cron: end'); + Logger::notice('cron: end'); DI::config()->set('system', 'last_cron', time()); From b1a9666e43af138e55389fb028fa9841f57763e5 Mon Sep 17 00:00:00 2001 From: Michael Date: Tue, 1 Sep 2020 08:30:12 +0000 Subject: [PATCH 3/6] Splitting workerqueue cleanup --- src/Worker/ClearWorkerqueue.php | 50 +++++++++++++++++++ src/Worker/Cron.php | 85 +++++++++++++-------------------- 2 files changed, 83 insertions(+), 52 deletions(-) create mode 100644 src/Worker/ClearWorkerqueue.php diff --git a/src/Worker/ClearWorkerqueue.php b/src/Worker/ClearWorkerqueue.php new file mode 100644 index 000000000..c46eba62f --- /dev/null +++ b/src/Worker/ClearWorkerqueue.php @@ -0,0 +1,50 @@ +. + * + */ + +namespace Friendica\Worker; + +use Friendica\Core\Worker; +use Friendica\Database\DBA; +use Friendica\DI; + +/** + * Delete all done workerqueue entries + */ +class ClearWorkerqueue +{ + public static function execute() + { + DBA::delete('workerqueue', ['`done` AND `executed` < UTC_TIMESTAMP() - INTERVAL 1 HOUR']); + + // Optimizing this table only last seconds + if (DI::config()->get('system', 'optimize_tables')) { + // We are acquiring the two locks from the worker to avoid locking problems + if (DI::lock()->acquire(Worker::LOCK_PROCESS, 10)) { + if (DI::lock()->acquire(Worker::LOCK_WORKER, 10)) { + DBA::e("OPTIMIZE TABLE `workerqueue`"); + DBA::e("OPTIMIZE TABLE `process`"); + DI::lock()->release(Worker::LOCK_WORKER); + } + DI::lock()->release(Worker::LOCK_PROCESS); + } + } + } +} diff --git a/src/Worker/Cron.php b/src/Worker/Cron.php index 3eb93ba26..4a23e86f9 100644 --- a/src/Worker/Cron.php +++ b/src/Worker/Cron.php @@ -24,9 +24,7 @@ namespace Friendica\Worker; use Friendica\Core\Hook; use Friendica\Core\Logger; use Friendica\Core\Worker; -use Friendica\Database\DBA; use Friendica\DI; -use Friendica\Util\DateTimeFormat; class Cron { @@ -46,11 +44,24 @@ class Cron } } - Logger::notice('cron: start'); + Logger::notice('start'); + + // Ensure to have a .htaccess file. + // this is a precaution for systems that update automatically + $basepath = $a->getBasePath(); + if (!file_exists($basepath . '/.htaccess') && is_writable($basepath)) { + copy($basepath . '/.htaccess-dist', $basepath . '/.htaccess'); + } // Fork the cron jobs in separate parts to avoid problems when one of them is crashing Hook::fork($a->queue['priority'], "cron"); + // Poll contacts + Worker::add(PRIORITY_HIGH, 'PollContacts'); + + // Update contact information + Worker::add(PRIORITY_LOW, 'UpdatePublicContacts'); + // run the process to update server directories in the background Worker::add(PRIORITY_LOW, 'UpdateServerDirectories'); @@ -63,12 +74,25 @@ class Cron // Repair entries in the database Worker::add(PRIORITY_LOW, 'RepairDatabase'); - // once daily run birthday_updates and then expire in background - $d1 = DI::config()->get('system', 'last_expire_day'); - $d2 = intval(DateTimeFormat::utcNow('d')); + // Hourly cron calls + if (DI::config()->get('system', 'last_cron_hourly', 0) + 3600 < time()) { + + // Search for new contacts in the directory + if (DI::config()->get('system', 'synchronize_directory')) { + Worker::add(PRIORITY_LOW, 'PullDirectory'); + } + + // Delete all done workerqueue entries + Worker::add(PRIORITY_LOW, 'ClearWorkerqueue'); + + // Clear cache entries + Worker::add(PRIORITY_LOW, 'ClearCache'); + + DI::config()->set('system', 'last_cron_hourly', time()); + } // Daily cron calls - if ($d2 != intval($d1)) { + if (DI::config()->get('system', 'last_cron_daily', 0) + 86400 < time()) { Worker::add(PRIORITY_LOW, 'UpdateContactBirthdays'); @@ -96,53 +120,10 @@ class Cron Worker::add(PRIORITY_LOW, 'OptimizeTables'); } - DI::config()->set('system', 'last_expire_day', $d2); + DI::config()->set('system', 'last_cron_daily', time()); } - // Hourly cron calls - if (DI::config()->get('system', 'last_cron_hourly', 0) + 3600 < time()) { - - // Search for new contacts in the directory - if (DI::config()->get('system', 'synchronize_directory')) { - Worker::add(PRIORITY_LOW, 'PullDirectory'); - } - - // Delete all done workerqueue entries - DBA::delete('workerqueue', ['`done` AND `executed` < UTC_TIMESTAMP() - INTERVAL 1 HOUR']); - - // Optimizing this table only last seconds - if (DI::config()->get('system', 'optimize_tables')) { - // We are acquiring the two locks from the worker to avoid locking problems - if (DI::lock()->acquire(Worker::LOCK_PROCESS, 10)) { - if (DI::lock()->acquire(Worker::LOCK_WORKER, 10)) { - DBA::e("OPTIMIZE TABLE `workerqueue`"); - DBA::e("OPTIMIZE TABLE `process`"); - DI::lock()->release(Worker::LOCK_WORKER); - } - DI::lock()->release(Worker::LOCK_PROCESS); - } - } - - // Clear cache entries - Worker::add(PRIORITY_LOW, 'ClearCache'); - - DI::config()->set('system', 'last_cron_hourly', time()); - } - - // Ensure to have a .htaccess file. - // this is a precaution for systems that update automatically - $basepath = $a->getBasePath(); - if (!file_exists($basepath . '/.htaccess') && is_writable($basepath)) { - copy($basepath . '/.htaccess-dist', $basepath . '/.htaccess'); - } - - // Poll contacts - Worker::add(PRIORITY_HIGH, 'PollContacts'); - - // Update contact information - Worker::add(PRIORITY_LOW, 'UpdatePublicContacts'); - - Logger::notice('cron: end'); + Logger::notice('end'); DI::config()->set('system', 'last_cron', time()); From 41e07a9ae28b9742de9a32b5a43a87ae9a5ffc32 Mon Sep 17 00:00:00 2001 From: Michael Date: Tue, 1 Sep 2020 08:32:53 +0000 Subject: [PATCH 4/6] Code cleanup --- src/Worker/Cron.php | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/Worker/Cron.php b/src/Worker/Cron.php index 4a23e86f9..cd6c2d39e 100644 --- a/src/Worker/Cron.php +++ b/src/Worker/Cron.php @@ -54,7 +54,7 @@ class Cron } // Fork the cron jobs in separate parts to avoid problems when one of them is crashing - Hook::fork($a->queue['priority'], "cron"); + Hook::fork($a->queue['priority'], 'cron'); // Poll contacts Worker::add(PRIORITY_HIGH, 'PollContacts'); @@ -126,7 +126,5 @@ class Cron Logger::notice('end'); DI::config()->set('system', 'last_cron', time()); - - return; } } From 3aa6d516bf611d5012ca1f4fc6521f1af1c46dc5 Mon Sep 17 00:00:00 2001 From: Michael Date: Tue, 1 Sep 2020 12:55:46 +0000 Subject: [PATCH 5/6] Changed priority --- src/Worker/Cron.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/Worker/Cron.php b/src/Worker/Cron.php index cd6c2d39e..2a2e5e57a 100644 --- a/src/Worker/Cron.php +++ b/src/Worker/Cron.php @@ -57,7 +57,7 @@ class Cron Hook::fork($a->queue['priority'], 'cron'); // Poll contacts - Worker::add(PRIORITY_HIGH, 'PollContacts'); + Worker::add(PRIORITY_MEDIUM, 'PollContacts'); // Update contact information Worker::add(PRIORITY_LOW, 'UpdatePublicContacts'); From 1c5801ca8e21906c4564cac0e1417f8458d2f5c0 Mon Sep 17 00:00:00 2001 From: Michael Date: Tue, 1 Sep 2020 13:40:37 +0000 Subject: [PATCH 6/6] Renamed classes --- .../{CheckdeletedContacts.php => CheckDeletedContacts.php} | 2 +- src/Worker/{ClearWorkerqueue.php => CleanWorkerqueue.php} | 2 +- src/Worker/Cron.php | 4 ++-- 3 files changed, 4 insertions(+), 4 deletions(-) rename src/Worker/{CheckdeletedContacts.php => CheckDeletedContacts.php} (97%) rename src/Worker/{ClearWorkerqueue.php => CleanWorkerqueue.php} (98%) diff --git a/src/Worker/CheckdeletedContacts.php b/src/Worker/CheckDeletedContacts.php similarity index 97% rename from src/Worker/CheckdeletedContacts.php rename to src/Worker/CheckDeletedContacts.php index c47495211..e1aa57fc7 100644 --- a/src/Worker/CheckdeletedContacts.php +++ b/src/Worker/CheckDeletedContacts.php @@ -28,7 +28,7 @@ use Friendica\Database\DBA; * Checks for contacts that are about to be deleted and ensures that they are removed. * This should be done automatically in the "remove" function. This here is a cleanup job. */ -class CheckdeletedContacts +class CheckDeletedContacts { public static function execute() { diff --git a/src/Worker/ClearWorkerqueue.php b/src/Worker/CleanWorkerqueue.php similarity index 98% rename from src/Worker/ClearWorkerqueue.php rename to src/Worker/CleanWorkerqueue.php index c46eba62f..00559f750 100644 --- a/src/Worker/ClearWorkerqueue.php +++ b/src/Worker/CleanWorkerqueue.php @@ -28,7 +28,7 @@ use Friendica\DI; /** * Delete all done workerqueue entries */ -class ClearWorkerqueue +class CleanWorkerQueue { public static function execute() { diff --git a/src/Worker/Cron.php b/src/Worker/Cron.php index 2a2e5e57a..7a23a2c70 100644 --- a/src/Worker/Cron.php +++ b/src/Worker/Cron.php @@ -83,7 +83,7 @@ class Cron } // Delete all done workerqueue entries - Worker::add(PRIORITY_LOW, 'ClearWorkerqueue'); + Worker::add(PRIORITY_LOW, 'CleanWorkerQueue'); // Clear cache entries Worker::add(PRIORITY_LOW, 'ClearCache'); @@ -114,7 +114,7 @@ class Cron // check upstream version? Worker::add(PRIORITY_LOW, 'CheckVersion'); - Worker::add(PRIORITY_LOW, 'CheckdeletedContacts'); + Worker::add(PRIORITY_LOW, 'CheckDeletedContacts'); if (DI::config()->get('system', 'optimize_tables')) { Worker::add(PRIORITY_LOW, 'OptimizeTables');