diff --git a/msgpush-module-push/msgpush-module-push-server/src/main/java/com/njcn/msgpush/module/push/dal/redis/MessageRetryRedisDAO.java b/msgpush-module-push/msgpush-module-push-server/src/main/java/com/njcn/msgpush/module/push/dal/redis/MessageRetryRedisDAO.java index b5578d0..c24cc3d 100644 --- a/msgpush-module-push/msgpush-module-push-server/src/main/java/com/njcn/msgpush/module/push/dal/redis/MessageRetryRedisDAO.java +++ b/msgpush-module-push/msgpush-module-push-server/src/main/java/com/njcn/msgpush/module/push/dal/redis/MessageRetryRedisDAO.java @@ -88,6 +88,11 @@ public class MessageRetryRedisDAO { * @param message 消息 */ public boolean addToRetryQueue(MessageRecordDO message) { + // 兜底守卫:nextRetryTime 为空则跳过单条,绝不让一条坏数据把整批/整个定时任务带崩 + if (message.getNextRetryTime() == null) { + log.warn("跳过入队:nextRetryTime 为空, messageId={}, channel={}", message.getId(), message.getChannel()); + return false; + } String key = getRetryQueueKey(message.getChannel()); double score = message.getNextRetryTime().atZone(ZoneId.systemDefault()).toInstant().toEpochMilli(); diff --git a/msgpush-module-push/msgpush-module-push-server/src/main/java/com/njcn/msgpush/module/push/service/retry/MessageRetryQueueServiceImpl.java b/msgpush-module-push/msgpush-module-push-server/src/main/java/com/njcn/msgpush/module/push/service/retry/MessageRetryQueueServiceImpl.java index 05bc800..fdbae57 100644 --- a/msgpush-module-push/msgpush-module-push-server/src/main/java/com/njcn/msgpush/module/push/service/retry/MessageRetryQueueServiceImpl.java +++ b/msgpush-module-push/msgpush-module-push-server/src/main/java/com/njcn/msgpush/module/push/service/retry/MessageRetryQueueServiceImpl.java @@ -28,6 +28,7 @@ import java.time.LocalDateTime; import java.util.Arrays; import java.util.Collections; import java.util.List; +import java.util.Map; import java.util.concurrent.atomic.AtomicBoolean; import java.util.stream.Collectors; @@ -310,6 +311,8 @@ public class MessageRetryQueueServiceImpl extends ServiceImpl messages, List queueRecords) { + Map nextRetryTimeMap = queueRecords.stream() + .filter(r -> r.getNextRetryTime() != null) + .collect(Collectors.toMap(MessageRetryQueueDO::getMessageId, MessageRetryQueueDO::getNextRetryTime, (a, b) -> a)); + messages.forEach(m -> { + LocalDateTime queueTime = nextRetryTimeMap.get(m.getId()); + if (queueTime != null) { + m.setNextRetryTime(queueTime); + } + }); + } + /** * 通过设置lockUntil时间、process_status=1来从数据库层抢占该条重试消息。 */