[executor] optimize logging

capcom6 · capcom6 · commit 8bbe45b7a244 · 2025-11-25T06:14:49.000+07:00
diff --git a/internal/worker/executor/service.go b/internal/worker/executor/service.go
@@ -100,12 +100,12 @@ func (s *Service) execute(ctx context.Context, task PeriodicTask) {
 	logger := s.logger.With(zap.String("name", task.Name()))
 
 	if err := s.locker.AcquireLock(ctx, task.Name()); err != nil {
-		logger.Error("failed to acquire lock", zap.String("name", task.Name()), zap.Error(err))
+		logger.Error("failed to acquire lock", zap.Error(err))
 		return
 	}
 	defer func() {
 		if err := s.locker.ReleaseLock(ctx, task.Name()); err != nil {
-			logger.Error("failed to release lock", zap.String("name", task.Name()), zap.Error(err))
+			logger.Error("failed to release lock", zap.Error(err))
 		}
 	}()
 

Original file line number	Diff line number	Diff line change
`@@ -100,12 +100,12 @@ func (s *Service) execute(ctx context.Context, task PeriodicTask) {`
`100`	`100`	`logger := s.logger.With(zap.String("name", task.Name()))`
`101`	`101`
`102`	`102`	`if err := s.locker.AcquireLock(ctx, task.Name()); err != nil {`
`103`		`- logger.Error("failed to acquire lock", zap.String("name", task.Name()), zap.Error(err))`
	`103`	`+ logger.Error("failed to acquire lock", zap.Error(err))`
`104`	`104`	`return`
`105`	`105`	`}`
`106`	`106`	`defer func() {`
`107`	`107`	`if err := s.locker.ReleaseLock(ctx, task.Name()); err != nil {`
`108`		`- logger.Error("failed to release lock", zap.String("name", task.Name()), zap.Error(err))`
	`108`	`+ logger.Error("failed to release lock", zap.Error(err))`
`109`	`109`	`}`
`110`	`110`	`}()`
`111`	`111`