- 去除OrderManager.getOrders中不必要的事务

business/jxcallback/orderman/orderman_ext.go

@@ -358,6 +358,7 @@ func (c *OrderManager) getOrders(ctx *jxcontext.Context, isIncludeSku bool, from
 			sqlParams = append(sqlParams, vendorIDs)
 		}
 	}
+	db := dao.GetDB()
 	sql += sqlWhere
 	if isIncludeSku {
 		sql += `
@@ -367,20 +368,21 @@ func (c *OrderManager) getOrders(ctx *jxcontext.Context, isIncludeSku bool, from
 		ORDER BY t1.order_created_at DESC
 		LIMIT ? OFFSET ?`
 		sqlParams = append(sqlParams, pageSize, offset)
-	}
 
-	db := dao.GetDB()
-	dao.Begin(db)
-	defer func() {
-		if r := recover(); r != nil {
-			dao.Rollback(db)
-			panic(r)
-		}
-	}()
+		dao.Begin(db)
+		defer func() {
+			if r := recover(); r != nil {
+				dao.Rollback(db)
+				panic(r)
+			}
+		}()
+	}
 	if err = dao.GetRows(db, &orders, sql, sqlParams...); err == nil {
 		totalCount = dao.GetLastTotalRowCount(db)
 	}
-	dao.Commit(db)
+	if !isIncludeSku {
+		dao.Commit(db)
+	}
 	return orders, totalCount, err
 }