liulei/jx-callback
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

- avoid memory leak in TransferLegacyJdOrder and TransferLegacyElmOrder.

Browse Source
This commit is contained in:
gazebo
2019-02-11 12:20:36 +08:00
parent dff3bf8316
commit 86f6d25849
1 changed files with 36 additions and 29 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
business/jxstore/tempop/tempop.go
View File

@@ -4,6 +4,7 @@ import (
"fmt" "fmt"
"math" "math"
"runtime" "runtime"
"strings"
"time" "time"
"git.rosy.net.cn/baseapi/platformapi/jdapi" "git.rosy.net.cn/baseapi/platformapi/jdapi"
@@ -289,8 +290,9 @@ func TransferLegacyJdOrder(ctx *jxcontext.Context, isAsync, isContinueWhenError
var orderDetailList []*model.GoodsOrderOriginal var orderDetailList []*model.GoodsOrderOriginal
for _, v := range batchItemList { for _, v := range batchItemList {
jdOrder := v.(*legacymodel2.Jdorder) jdOrder := v.(*legacymodel2.Jdorder)
if len(jdOrder.Data) > 10 {
var detail map[string]interface{} var detail map[string]interface{}
if err = utils.UnmarshalUseNumber([]byte(jdOrder.Data), &detail); err != nil { if err = utils.UnmarshalUseNumber([]byte(strings.Replace(strings.Replace(jdOrder.Data, "\n", "", -1), "\r", "", -1)), &detail); err != nil {
return nil, err return nil, err
} }
resultList := detail["result"].(map[string]interface{})["resultList"].([]interface{}) resultList := detail["result"].(map[string]interface{})["resultList"].([]interface{})
@@ -309,12 +311,14 @@ func TransferLegacyJdOrder(ctx *jxcontext.Context, isAsync, isContinueWhenError
} }
} }
} }
}
if len(orderDetailList) > 0 { if len(orderDetailList) > 0 {
err = dao.CreateMultiEntities(db, orderDetailList) err = dao.CreateMultiEntities(db, orderDetailList)
} }
return nil, err return nil, err
}, jdOrderList) }, jdOrderList)
rootTask.AddChild(task).Run() // rootTask.AddChild(task).Run()
task.Run()
_, err = task.GetResult(0) _, err = task.GetResult(0)
runtime.GC() runtime.GC()
} else { } else {
@@ -352,8 +356,9 @@ func TransferLegacyElmOrder(ctx *jxcontext.Context, isAsync, isContinueWhenError
var orderDetailList []*model.GoodsOrderOriginal var orderDetailList []*model.GoodsOrderOriginal
for _, v := range batchItemList { for _, v := range batchItemList {
elmOrder := v.(*legacymodel2.Elemeorder) elmOrder := v.(*legacymodel2.Elemeorder)
if len(elmOrder.Data) > 10 {
var detail map[string]interface{} var detail map[string]interface{}
if err = utils.UnmarshalUseNumber([]byte(elmOrder.Data), &detail); err != nil { if err = utils.UnmarshalUseNumber([]byte(strings.Replace(strings.Replace(elmOrder.Data, "\n", "", -1), "\r", "", -1)), &detail); err != nil {
return nil, err return nil, err
} }
orderDetail := &model.GoodsOrderOriginal{ orderDetail := &model.GoodsOrderOriginal{
@@ -365,12 +370,14 @@ func TransferLegacyElmOrder(ctx *jxcontext.Context, isAsync, isContinueWhenError
} }
orderDetailList = append(orderDetailList, orderDetail) orderDetailList = append(orderDetailList, orderDetail)
} }
}
if len(orderDetailList) > 0 { if len(orderDetailList) > 0 {
err = dao.CreateMultiEntities(db, orderDetailList) err = dao.CreateMultiEntities(db, orderDetailList)
} }
return nil, err return nil, err
}, elmOrderList) }, elmOrderList)
rootTask.AddChild(task).Run() // rootTask.AddChild(task).Run()
task.Run()
_, err = task.GetResult(0) _, err = task.GetResult(0)
runtime.GC() runtime.GC()
} else { } else {