From abfd401180d1eb09c8ed1c24797c3e503e45fa08 Mon Sep 17 00:00:00 2001
From: liuxiaolong <liuxiaolong@aiotlink.com>
Date: 星期日, 07 二月 2021 15:02:50 +0800
Subject: [PATCH] 调整超时为3s
---
hbusc.go | 184 +++++++++++++++++++++++++++++----------------
1 files changed, 119 insertions(+), 65 deletions(-)
diff --git a/hbusc.go b/hbusc.go
index 180a929..640b55c 100644
--- a/hbusc.go
+++ b/hbusc.go
@@ -7,7 +7,6 @@
"errors"
"fmt"
"os"
- "strconv"
"sync"
"time"
)
@@ -48,10 +47,9 @@
sockSub *sockClient //璁㈤槄涓婚鐨剆ocket锛岀嚎绋嬪疄鏃舵帴鏀舵秷鎭紝闇�瑕佸崟鐙鐞�
sockWorker *sockClient //鍙戠粰浠绘剰鐨剆erver,鐭殏鐨剅equest client
- mtxWorker sync.Mutex //SendAndRecv鍙兘涓嶆槸绾跨▼瀹夊叏鐨�
+ //mtxWorker sync.Mutex //SendAndRecv鍙兘涓嶆槸绾跨▼瀹夊叏鐨�
chSub chan TransInfo
- chReply chan TransInfo
}
//鑾峰彇鍏朵粬杩涚▼鍙戠粰姝ocket鐨勬秷鎭�,鍙互鏄痵ub鐨勬帴鏀讹紝涔熷彲浠ユ槸reply鐨勬帴鏀躲��
@@ -65,7 +63,8 @@
wg.Done()
return
default:
- if n := s.RecvfromTimeout(&data, &key, 1000);n == 0 {
+ n := s.RecvfromTimeout(&data, &key, 1000) //鐩墠10001杩斿洖鍊艰〃绀鸿秴鏃�
+ if n == 0 {
var info MsgInfo
if err := json.Unmarshal(data, &info);err == nil {
ch <- TransInfo{
@@ -75,9 +74,29 @@
data = []byte{}
key = 0
+ } else {
+ logFn("unmarshal to MsgInfo err:", err)
}
} else {
- time.Sleep(10 * time.Millisecond)
+ time.Sleep(100 * time.Millisecond)
+ }
+ }
+ }
+}
+
+func recvandsendRoutine(ctx context.Context, s *bhomebus.Socket, wg *sync.WaitGroup,serveFn func(rdata []byte, rkey int, sdata *[]byte) bool, logFn func(...interface{})) {
+ for {
+ select {
+ case <-ctx.Done():
+ logFn("recvandsendRoutine ctx.Done")
+ wg.Done()
+ return
+ default:
+ n := s.RecvandsendTimeout(1000, serveFn) //鐩墠10001杩斿洖鍊艰〃绀鸿秴鏃�
+ if n != 0 {
+ logFn("RecvandsendTimeout success")
+ } else {
+ //time.Sleep(100 * time.Millisecond)
}
}
}
@@ -85,11 +104,11 @@
//Register
func Register(ctx context.Context, q chan os.Signal, config *Config, ri *RegisterInfo) (*BHBus,error) {
- handle := &BHBus{
+ handle := &BHBus {
+ ctx: ctx,
conf: config,
m: make(map[string]*sockServer),
chSub: make(chan TransInfo, config.chSize),
- chReply: make(chan TransInfo, config.chSize),
}
var err error
@@ -151,30 +170,32 @@
}
var rMsg []bhomebus.Mesg
- n := regSock.Sendandrecv(regAddr, msg, &rMsg) //n浠h〃鎴愬姛鍙戦�佺殑鑺傜偣鐨勪釜鏁�
- handle.printLog("regSock.Sendandrecv n:", n, "len(rMsg):", len(rMsg))
+ n := regSock.SendandrecvTimeout(regAddr, msg, &rMsg, handle.conf.sendTimeOut) //n浠h〃鎴愬姛鍙戦�佺殑鑺傜偣鐨勪釜鏁�
+ handle.printLog("regSock.SendandrecvTimeout n:", n)
if n == 1 && len(rMsg) == 1 {
- var cr CommonReply
+ var cr Reply
if err = json.Unmarshal(rMsg[0].Data, &cr);err != nil {
handle.printLog("unmarshal regReply err:", err)
return nil, errors.New("unmarshal regReply err:"+err.Error())
} else {
- if cr.Status == REPLY_SUCCESS {
- var rr RegisterReply
- if err = json.Unmarshal(cr.Body, &rr);err ==nil {
- regR = &rr
- break loop
+ if cr.Success {
+ if rpd,err := json.Marshal(cr.Data);err ==nil {
+ var rr RegisterReply
+ if err = json.Unmarshal(rpd, &rr); err == nil {
+ regR = &rr
+ break loop
+ } else {
+ handle.printLog("unmarshal RegisterReply err:", err)
+ }
} else {
- handle.printLog("unmarshal RegisterReply err:", err)
+ handle.printLog("marshal cr.Data err:", err)
}
-
} else {
- handle.printLog("cr.Status:", cr.Status, "Desc:", cr.Desc)
+ handle.printLog("cr:", cr)
}
-
}
} else {
- time.Sleep(100 * time.Millisecond)
+ time.Sleep(1 * time.Second)
}
}
}
@@ -194,6 +215,7 @@
//缁存寔蹇冭烦鐨剆ocket
sockHB := bhomebus.OpenSocket()
+ handle.printLog("open sockHB")
handle.sockHB = &sockClient{
sock: sockHB,
peer: int(regR.HeartbeatKey),
@@ -204,9 +226,10 @@
if ri.PubTopic != nil && len(ri.PubTopic) > 0 {
sockReply := bhomebus.OpenSocket()
sockReply.ForceBind(int(regR.ReplyKey))
- handle.wg.Add(1)
+ handle.printLog("after pubTopic forceBind")
+ //handle.wg.Add(1)
//serve server reply
- go recvRoutine(ctx, sockReply, handle.wg, handle.chReply, handle.printLog)
+ //go recvRoutine(ctx, sockReply, handle.wg, handle.chReply, handle.printLog)
handle.sockRep = &sockServer{
sock: sockReply,
info: &ri.Proc,
@@ -226,8 +249,10 @@
//璁㈤槄娑堟伅鐨剆ocket
sockSub := bhomebus.OpenSocket()
//璁㈤槄鎵�鏈変富棰�
+ handle.printLog("start Sub topics")
for _,v := range ri.SubTopic {
- sockSub.Sub(v)
+ subN := sockSub.Sub(v)
+ handle.printLog("subTopic:", v, " ret n:", subN)
}
//鍚姩璁㈤槄淇℃伅鎺ユ敹
@@ -264,8 +289,8 @@
return err
}
- h.mtxWorker.Lock()
- defer h.mtxWorker.Unlock()
+ //h.mtxWorker.Lock()
+ //defer h.mtxWorker.Unlock()
netNodes := append([]bhomebus.NetNode{}, bhomebus.NetNode{
Key: h.conf.regKey,
})
@@ -385,34 +410,40 @@
var ret []bhomebus.Mesg
n := h.sockWorker.sock.SendandrecvTimeout(reqNetNode, reqD, &ret, h.conf.sendTimeOut)
if n > 0 {
- var reply CommonReply
+ var reply Reply
err = json.Unmarshal(ret[0].Data, &reply)
if err != nil {
h.printLog("unmarshal err:", err)
return nil, err
}
- if reply.Status == REPLY_SUCCESS {
- err = json.Unmarshal(reply.Body, &nodes)
+ if reply.Success {
+ rd,err := json.Marshal(reply.Data)
if err == nil {
- return nodes, nil
+ err = json.Unmarshal(rd, &nodes)
+ if err == nil {
+ return nodes, nil
+ } else {
+ h.printLog("unmarshal err:", err, "nodes:", nodes, " center reply.Data:", reply.Data)
+ return nil, fmt.Errorf("unmarshal to nodes err:%s", err.Error())
+ }
} else {
- h.printLog("unmarshal err:", err, "nodes:", nodes)
- return nil, fmt.Errorf("unmarshal reply.Body err:%s", err.Error())
+ return nil, fmt.Errorf("marshal reply.Data err:%s", err.Error())
}
+
} else {
- h.printLog("reply status:", reply.Status, "desc:", reply.Desc, "body:", string(reply.Body))
- return nil, fmt.Errorf("REPLY STATUS:%d", reply.Status)
+ h.printLog("reply success:", reply.Success, "msg:", reply.Msg, "data:", reply.Data)
+ return nil, fmt.Errorf("REPLY msg:%s", reply.Msg)
}
} else {
return nil, fmt.Errorf("GetNetNodeByTopic ret n:%d", n)
}
}
-func (h *BHBus) Request(serverId string, req *MsgInfo, milliSecs int) (resp *MsgInfo, err error) {
+func (h *BHBus) Request(serverId string, req *MsgInfo, milliSecs int) (*Reply, error) {
//1.棣栧厛闇�瑕侀�氳繃topic鎷垮埌鏈満瀵瑰簲鐨凬etNode
rNodes, err := h.GetNetNodeByTopic(serverId, &req.SrcProc, req.Topic)
- h.printLog("rNodes:", rNodes, "err:", err)
+ h.printLog("topic", req.Topic, "rNodes:", rNodes, "err:", err)
if err != nil {
return nil, err
}
@@ -424,28 +455,48 @@
}
var ret []bhomebus.Mesg
- if n := h.sockWorker.sock.SendandrecvTimeout(rNodes, data,&ret, milliSecs);n > 0 {
- if len(ret) > 0 {
- if err = json.Unmarshal(ret[0].Data, resp); err == nil {
- return resp, nil
- }
+ n := h.sockWorker.sock.SendandrecvTimeout(rNodes, data,&ret, milliSecs)
+
+ if n > 0 && len(ret) > 0 {
+ var resp Reply
+ if err = json.Unmarshal(ret[0].Data, &resp); err == nil {
+ return &resp, nil
+ } else {
+ h.printLog("unmarshal ret[0].Data err:", err)
+ return nil, err
}
+ } else {
+ h.printLog("Request n: ", n, " len(ret): ", len(ret))
}
return nil, fmt.Errorf("request err")
}
-func (h *BHBus) Reply(replyKey int, i MsgInfo) error {
- data,err := json.Marshal(i)
- if err != nil {
- return err
- }
+func (h *BHBus) RequestOnly(rData []byte, rNodes []bhomebus.NetNode) ([]byte, error) {
+ var ret []bhomebus.Mesg
- n := h.sockRep.sock.SendtoTimeout(data, replyKey, h.conf.sendTimeOut)
- if n != 0 {
- return errors.New("reply sendToTimeOut n:"+strconv.Itoa(n))
+ n := h.sockWorker.sock.SendandrecvTimeout(rNodes, rData, &ret, h.conf.sendTimeOut)
+
+ if n > 0 && len(ret) > 0 {
+ return ret[0].Data, nil
+ } else {
+ h.printLog("Request n: ", n, " len(ret): ", len(ret))
}
- return nil
+ return nil, fmt.Errorf("request err")
}
+
+//func (h *BHBus) Reply(replyKey int, i *Reply) error {
+// data,err := json.Marshal(*i)
+// if err != nil {
+// return err
+// }
+//
+// n := h.sockRep.sock.SendtoTimeout(data, replyKey, h.conf.sendTimeOut)
+// h.printLog("reply to key:", replyKey, " n:",n)
+// if n != 0 {
+// return errors.New("reply sendToTimeOut n:"+strconv.Itoa(n))
+// }
+// return nil
+//}
//鍙彂閫佽姹傦紝涓嶉渶瑕佸簲绛�.
@@ -455,8 +506,8 @@
if err != nil {
return err
}
- h.mtxWorker.Lock()
- defer h.mtxWorker.Unlock()
+ //h.mtxWorker.Lock()
+ //defer h.mtxWorker.Unlock()
n := h.sockWorker.sock.SendtoTimeout(data, key, h.conf.sendTimeOut)
if n != 0 {
return fmt.Errorf("sendOnly ret n:%d", n)
@@ -464,7 +515,7 @@
return nil
}
-func (h *BHBus) RequestCenter(req *MsgInfo) (*CommonReply, error) {
+func (h *BHBus) RequestCenter(req *MsgInfo) (*Reply, error) {
data, err := json.Marshal(*req)
if err != nil {
return nil, err
@@ -472,13 +523,13 @@
rNodes := append([]bhomebus.NetNode{}, bhomebus.NetNode{
Key: KEY_QUERY,
})
- h.mtxWorker.Lock()
- defer h.mtxWorker.Unlock()
+ //h.mtxWorker.Lock()
+ //defer h.mtxWorker.Unlock()
var ret []bhomebus.Mesg
n := h.sockWorker.sock.SendandrecvTimeout(rNodes, data,&ret, h.conf.sendTimeOut)
h.printLog("requestCenter n:", n, "len(ret):", len(ret))
if n > 0 && len(ret) > 0{
- var cr CommonReply
+ var cr Reply
if err = json.Unmarshal(ret[0].Data, &cr); err == nil {
return &cr, nil
} else {
@@ -493,7 +544,7 @@
func (h *BHBus) Pub(nodes []bhomebus.NetNode, msg *MsgInfo) error {
data,err := json.Marshal(*msg)
if err == nil {
- if n := h.sockPub.sock.PubTimeout(nodes, msg.Topic, data, h.conf.pubTimeOut);n == 0 {
+ if n := h.sockPub.sock.PubTimeout(nodes, msg.Topic, data, h.conf.pubTimeOut);n > 0 {
return nil
} else {
return fmt.Errorf("pub err n:%d", n)
@@ -501,6 +552,14 @@
}
return err
+}
+
+func (h *BHBus) PubTimeout(nodes []bhomebus.NetNode, msg *MsgInfo, timeout int) int {
+ data,err := json.Marshal(*msg)
+ if err == nil {
+ return h.sockPub.sock.PubTimeout(nodes, msg.Topic, data, timeout)
+ }
+ return -1
}
//杩藉姞璁㈤槄鐨勪富棰樻秷鎭�
@@ -525,18 +584,13 @@
//鑾峰彇sub 鎴栬�呴渶瑕乺eply鐨勬秷鎭�
-func (h *BHBus) GetMsg() (subMsg *MsgInfo, replyMsg *MsgInfo, replyKey int) {
- if h.sockHB == nil && h.sockRep == nil && h.sockPub == nil && h.sockSub == nil && h.sockWorker == nil {
- return nil,nil, -1
+func (h *BHBus) GetMsg() (subMsg *MsgInfo) {
+ if h.sockHB == nil && h.sockPub == nil && h.sockSub == nil && h.sockWorker == nil {
+ return nil
}
- if len(h.chSub) >1 {
+ if len(h.chSub) >0 {
m := <-h.chSub
subMsg = m.info
- }
- if len(h.chReply) > 1 {
- m := <-h.chReply
- replyMsg = m.info
- replyKey = m.port
}
return
}
\ No newline at end of file
--
Gitblit v1.8.0