Merge pull request #1630 from appwrite/fix-deletes-worker-usage-constant
fix(refactor-db): use correct constant when deleting usage stats
This commit is contained in:
commit
45bf9a20ae
3 changed files with 8 additions and 2 deletions
|
@ -35,6 +35,12 @@ install:
|
|||
|
||||
script:
|
||||
- docker ps -a
|
||||
# Tests should fail if any container is in exited status
|
||||
- ALL_UP=`docker ps -aq --filter "status=exited"`
|
||||
- >
|
||||
if [[ "$ALL_UP" != "" ]]; then
|
||||
exit 1
|
||||
fi
|
||||
- docker-compose logs appwrite
|
||||
- docker-compose logs mariadb
|
||||
- docker-compose logs appwrite-worker-functions
|
||||
|
|
|
@ -42,7 +42,7 @@ $cli
|
|||
function notifyDeleteUsageStats(int $interval30m, int $interval1d)
|
||||
{
|
||||
Resque::enqueue(Event::DELETE_QUEUE_NAME, Event::DELETE_CLASS_NAME, [
|
||||
'type' => DELETE_TYPE_USAGE_STATS,
|
||||
'type' => DELETE_TYPE_USAGE,
|
||||
'timestamp1d' => time() - $interval1d,
|
||||
'timestamp30m' => time() - $interval30m,
|
||||
]);
|
||||
|
|
|
@ -79,7 +79,7 @@ class DeletesV1 extends Worker
|
|||
$this->deleteCertificates($document);
|
||||
break;
|
||||
|
||||
case DELETE_TYPE_USAGE_STATS:
|
||||
case DELETE_TYPE_USAGE:
|
||||
$this->deleteUsageStats($this->args['timestamp1d'], $this->args['timestamp30m']);
|
||||
break;
|
||||
default:
|
||||
|
|
Loading…
Reference in a new issue