From 58ca88b5008448532f19dca8bf030ceeba347bd5 Mon Sep 17 00:00:00 2001
From: zwl <1051256694@qq.com>
Date: 星期二, 28 四月 2026 19:05:32 +0800
Subject: [PATCH] 修复移库任务下发给wcs
---
src/main/java/com/zy/asrs/task/WorkMastScheduler.java | 148 +++++++++++++++++++++++++++++++++++--------------
1 files changed, 105 insertions(+), 43 deletions(-)
diff --git a/src/main/java/com/zy/asrs/task/WorkMastScheduler.java b/src/main/java/com/zy/asrs/task/WorkMastScheduler.java
index d1dc7cc..4f89dbc 100644
--- a/src/main/java/com/zy/asrs/task/WorkMastScheduler.java
+++ b/src/main/java/com/zy/asrs/task/WorkMastScheduler.java
@@ -9,7 +9,10 @@
import com.zy.asrs.service.WrkMastService;
import com.zy.asrs.task.core.ReturnT;
import com.zy.asrs.task.handler.WorkMastHandler;
+import com.zy.asrs.task.support.OutboundBatchSeqReleaseGuard;
+import com.zy.asrs.task.support.WorkPublishLockKeys;
import com.zy.asrs.utils.Utils;
+import com.zy.common.utils.RedisUtil;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import org.springframework.beans.factory.annotation.Autowired;
@@ -32,8 +35,8 @@
public class WorkMastScheduler {
private static final Logger log = LoggerFactory.getLogger(WorkMastScheduler.class);
- private static final long OUT_LOCK_REPORT_SUCCESS_WRK_STS = 21L;
- private static final long OUT_LOCK_REPORT_FAIL_WRK_STS = 22L;
+ private static final int MAX_PUBLISH_TASKS_ONCE = 20;
+ private static final long OUTBOUND_USER_NO_LOCK_SECONDS = 60L;
@Autowired
private WcsApiService wcsApiService;
@@ -41,6 +44,10 @@
private WrkMastService wrkMastService;
@Autowired
private WorkMastHandler workMastHandler;
+ @Autowired
+ private OutboundBatchSeqReleaseGuard outboundBatchSeqReleaseGuard;
+ @Autowired
+ private RedisUtil redisUtil;
@Scheduled(cron = "0/3 * * * * ? ")
private void execute(){
@@ -67,15 +74,16 @@
* 璋冨害鍣ㄥ彧璐熻矗浠庡伐浣滄。涓寫鍑衡�滃綋鍓嶅厑璁镐笅鍙戔�濈殑浠诲姟锛屽苟灏嗗叾杞崲鎴� WCS 鎺ュ彛闇�瑕佺殑鎶ユ枃缁撴瀯锛�
* 鍑哄簱浠诲姟鎸� userNo -> batchSeq 鍒嗗眰姹囨�诲悗涓茶涓嬪彂锛岀‘淇濆悓涓� userNo 涓嬪墠涓�涓� batchSeq 瀹屾垚鍚庡啀鍙戜笅涓�涓��
* <p>
- * 褰撳墠鎵归噺涓嬪彂鐨勫綊骞剁淮搴︽槸锛�
- * 1. WCS鎺ュ彛璺緞锛堝叆搴�/鍑哄簱/绉诲簱涓嶈兘娣峰彂锛夛紱
- * 2. 鍑哄簱浠诲姟鎸� work_mast.user_no -> work_mast.batch_seq 鍒嗗眰姹囨�伙紝骞舵寜 batchSeq 鑷劧鍗囧簭涓嬪彂銆�
+ * 褰撳墠涓嬪彂瑙勫垯鏄細
+ * 1. 鍏ュ簱浠诲姟鎸� WCS 鎺ュ彛鎵归噺涓嬪彂锛�
+ * 2. 绉诲簱浠诲姟鎸夊崟鏉′换鍔′笅鍙戯紱
+ * 3. 鍑哄簱浠诲姟鎸� work_mast.user_no -> work_mast.batch_seq 鍒嗗眰姹囨�伙紝骞舵寜 batchSeq 鑷劧鍗囧簭涓嬪彂銆�
*
* @author Ryan
* @date 2026/1/10 14:42
*/
@Scheduled(cron = "0/10 * * * * ? ")
- private void autoPubTasks() {
+ private synchronized void autoPubTasks() {
// 浠呭鐞嗗緟涓嬪彂/宸茬敓鎴愪笅鍙戝彿鐨勫伐浣滄。銆�
List<WrkMast> wrkMasts = wrkMastService.selectList(new EntityWrapper<WrkMast>().in("wrk_sts", Arrays.asList(1L, 11L))
.orderBy("user_no", true)
@@ -86,6 +94,7 @@
}
List<WorkTaskParams> paramsList = new ArrayList<>();
+ List<WorkTaskParams> moveParamsList = new ArrayList<>();
Map<String, LinkedHashMap<String, List<WorkTaskParams>>> outboundTasksByUserNo = new LinkedHashMap<>();
for (WrkMast wrkMast : wrkMasts) {
// 鍑哄簱绫讳换鍔★紙ioType > 100锛夐粯璁ら渶瑕� ERP 纭锛涙湭纭鐨勪换鍔″湪杩欓噷鐩存帴璺宠繃銆�
@@ -93,8 +102,13 @@
continue;
}
- WorkTaskParams params = buildWorkTaskParams(wrkMast);
if (isOutboundPublishTask(wrkMast)) {
+ if (Cools.isEmpty(wrkMast.getBatchSeq())) {
+ log.warn("鍑哄簱杩涗粨缂栧彿(batchSeq)涓虹┖锛岃烦杩囦笅鍙�, wrkNo={}, userNo={}",
+ wrkMast.getWrkNo(), wrkMast.getUserNo());
+ continue;
+ }
+ WorkTaskParams params = buildWorkTaskParams(wrkMast);
String userNo = normalizeGroupKey(wrkMast.getUserNo());
String batchSeq = normalizeGroupKey(wrkMast.getBatchSeq());
outboundTasksByUserNo
@@ -102,15 +116,21 @@
.computeIfAbsent(batchSeq, key -> new ArrayList<>())
.add(params);
} else {
- paramsList.add(params);
+ WorkTaskParams params = buildWorkTaskParams(wrkMast);
+ if (isMovePublishTask(params)) {
+ moveParamsList.add(params);
+ } else {
+ paramsList.add(params);
+ }
}
}
- if (!paramsList.isEmpty()) {
- R r = wcsApiService.pubWrksToWcs(paramsList);
- if (r == null || !Objects.equals(r.get("code"), 200)) {
- log.warn("鎵归噺涓嬪彂浠诲姟鍒癢CS澶辫触, result={}", r);
- }
+ if (publishTaskChunks(paramsList)) {
+ return;
+ }
+
+ if (publishMoveTasksOneByOne(moveParamsList)) {
+ return;
}
if (outboundTasksByUserNo.isEmpty()) {
@@ -123,10 +143,9 @@
batchSeqs.sort(this::compareBatchSeqNatural);
for (String batchSeq : batchSeqs) {
- String blockingBatchSeq = findFirstUnfinishedOutboundBatchSeq(userNo);
- if (blockingBatchSeq != null && compareBatchSeqNatural(batchSeq, blockingBatchSeq) != 0) {
- log.info("鍑哄簱鎵规鏈畬鎴愶紝鏆傚仠鍚庣画涓嬪彂, userNo={}, blockingBatchSeq={}, nextBatchSeq={}",
- userNo, blockingBatchSeq, batchSeq);
+ String blockMsg = outboundBatchSeqReleaseGuard.validateReady(userNo, batchSeq);
+ if (!Cools.isEmpty(blockMsg)) {
+ log.info(blockMsg);
break;
}
@@ -135,10 +154,8 @@
continue;
}
- R r = wcsApiService.pubWrksToWcs(batchParams);
- if (r == null || !Objects.equals(r.get("code"), 200)) {
- log.warn("鎵归噺涓嬪彂鍑哄簱浠诲姟鍒癢CS澶辫触, userNo={}, batchSeq={}, result={}", userNo, batchSeq, r);
- break;
+ if (publishOutboundTaskChunks(userNo, batchSeq, batchParams)) {
+ return;
}
}
}
@@ -185,28 +202,8 @@
return wrkMast != null && Objects.equals(wrkMast.getIoType(), 101);
}
- private String findFirstUnfinishedOutboundBatchSeq(String userNo) {
- EntityWrapper<WrkMast> wrapper = new EntityWrapper<>();
- if (Cools.isEmpty(userNo)) {
- wrapper.isNull("user_no");
- } else {
- wrapper.eq("user_no", userNo);
- }
- wrapper.eq("io_type", 101);
- wrapper.last(" and (wrk_sts < 14 or wrk_sts in ("
- + OUT_LOCK_REPORT_SUCCESS_WRK_STS + "," + OUT_LOCK_REPORT_FAIL_WRK_STS + "))");
- List<WrkMast> rows = wrkMastService.selectList(wrapper);
- if (rows == null || rows.isEmpty()) {
- return null;
- }
- String firstBatchSeq = null;
- for (WrkMast row : rows) {
- String batchSeq = normalizeGroupKey(row.getBatchSeq());
- if (firstBatchSeq == null || compareBatchSeqNatural(batchSeq, firstBatchSeq) < 0) {
- firstBatchSeq = batchSeq;
- }
- }
- return firstBatchSeq;
+ private boolean isMovePublishTask(WorkTaskParams params) {
+ return params != null && "move".equalsIgnoreCase(params.getType());
}
private int compareBatchSeqNatural(String left, String right) {
@@ -252,4 +249,69 @@
return Cools.isEmpty(value) ? "" : value;
}
+ private boolean publishTaskChunks(List<WorkTaskParams> paramsList) {
+ if (paramsList == null || paramsList.isEmpty()) {
+ return false;
+ }
+ for (int start = 0; start < paramsList.size(); start += MAX_PUBLISH_TASKS_ONCE) {
+ int end = Math.min(start + MAX_PUBLISH_TASKS_ONCE, paramsList.size());
+ List<WorkTaskParams> chunk = paramsList.subList(start, end);
+ R r = wcsApiService.pubWrksToWcs(chunk);
+ if (isWcsSuccess(r)) {
+ return true;
+ }
+ log.warn("鎵归噺涓嬪彂浠诲姟鍒癢CS澶辫触, start={}, size={}, result={}", start, chunk.size(), r);
+ }
+ return false;
+ }
+
+ private boolean publishMoveTasksOneByOne(List<WorkTaskParams> moveParamsList) {
+ if (moveParamsList == null || moveParamsList.isEmpty()) {
+ return false;
+ }
+ for (WorkTaskParams params : moveParamsList) {
+ R r = wcsApiService.pubWrkToWcs(params);
+ if (isWcsSuccess(r)) {
+ return true;
+ }
+ log.warn("绉诲簱浠诲姟涓嬪彂鍒癢CS澶辫触, taskNo={}, result={}",
+ params == null ? null : params.getTaskNo(), r);
+ }
+ return false;
+ }
+
+ private boolean publishOutboundTaskChunks(String userNo, String batchSeq, List<WorkTaskParams> batchParams) {
+ if (batchParams == null || batchParams.isEmpty()) {
+ return false;
+ }
+ for (int start = 0; start < batchParams.size(); start += MAX_PUBLISH_TASKS_ONCE) {
+ int end = Math.min(start + MAX_PUBLISH_TASKS_ONCE, batchParams.size());
+ List<WorkTaskParams> chunk = batchParams.subList(start, end);
+ String lockKey = WorkPublishLockKeys.outboundUserNoLock(userNo);
+ String lockValue = String.valueOf(System.currentTimeMillis());
+ if (!redisUtil.setIfAbsent(lockKey, lockValue, OUTBOUND_USER_NO_LOCK_SECONDS)) {
+ log.info("鍑哄簱浠诲姟姝ｅ湪涓嬪彂锛岃烦杩囨湰杞�, userNo={}, batchSeq={}, lockKey={}", userNo, batchSeq, lockKey);
+ return false;
+ }
+ try {
+ R r = wcsApiService.pubWrksToWcs(chunk);
+ if (isWcsSuccess(r)) {
+ return true;
+ }
+ log.warn("鎵归噺涓嬪彂鍑哄簱浠诲姟鍒癢CS澶辫触, userNo={}, batchSeq={}, start={}, size={}, result={}",
+ userNo, batchSeq, start, chunk.size(), r);
+ } finally {
+ Object currentLockValue = redisUtil.get(lockKey);
+ if (Objects.equals(currentLockValue, lockValue)) {
+ redisUtil.del(lockKey);
+ }
+ }
+ }
+ return false;
+ }
+
+ private boolean isWcsSuccess(R r) {
+ return r != null && Objects.equals(r.get("code"), 200);
+ }
+
}
--
Gitblit v1.9.1