Просмотр исходного кода

refactor: Handle nits from coderabbit

Will Nahmens 4 месяцев назад
Родитель
Сommit
3291d16de3

+ 1 - 1
packages/core/src/plugin/default-scheduler-plugin/default-scheduler-strategy.ts

@@ -10,7 +10,7 @@ import { TransactionalConnection } from '../../connection';
 import { ProcessContext } from '../../process-context';
 import { ScheduledTask } from '../../scheduler/scheduled-task';
 import { SchedulerStrategy, TaskReport } from '../../scheduler/scheduler-strategy';
-import { TaskService } from '../../service/services/task.service';
+import { TaskService } from '../../service';
 
 import { DEFAULT_SCHEDULER_PLUGIN_OPTIONS } from './constants';
 import { ScheduledTaskRecord } from './scheduled-task-record.entity';

+ 5 - 1
packages/core/src/service/services/task.service.ts

@@ -53,6 +53,7 @@ export class TaskService {
         return this.connection.rawConnection
             .getRepository(ScheduledTaskRecord)
             .createQueryBuilder('task')
+            .select(['task.taskId', 'task.lockedAt'])
             .where('task.lockedAt IS NOT NULL')
             .getMany();
     }
@@ -126,7 +127,10 @@ export class TaskService {
                     { lockedAt: null },
                 );
                 if (result.affected && result.affected > 0) {
-                    Logger.verbose(`Successfully cleaned stale task locks for task "${task.taskId}"`);
+                    Logger.verbose(
+                        `Successfully cleaned stale task locks for task "${task.taskId}"`,
+                        loggerCtx,
+                    );
                 } else {
                     Logger.debug(
                         `Skipped clearing lock for task "${task.taskId}" because the lock has changed since observation`,