waitInvoke rename waitResp

TarsCloud · Apr 26, 2023 · 4f7a65a · 4f7a65a
1 parent a415c7b
commit 4f7a65a
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 12 deletions.
diff --git a/tars/message.go b/tars/message.go
@@ -38,6 +38,7 @@ type Message struct {
 	isHash   bool
 	Async    bool
 	Callback model.Callback
+	RespCh   chan *requestf.ResponsePacket
 }
 
 // Init define the beginTime

diff --git a/tars/servant.go b/tars/servant.go
@@ -129,9 +129,7 @@ func (s *ServantProxy) TarsInvoke(ctx context.Context, cType byte,
 
 	msg := buildMessage(ctx, cType, sFuncName, buf, status, reqContext, resp, s)
 	timeout := time.Duration(s.syncTimeout) * time.Millisecond
-	err := s.invokeFilters(ctx, msg, timeout)
-
-	if err != nil {
+	if err := s.invokeFilters(ctx, msg, timeout); err != nil {
 		return err
 	}
 	*resp = *msg.Resp
@@ -238,8 +236,8 @@ func (s *ServantProxy) doInvoke(ctx context.Context, msg *Message, timeout time.
 	}
 
 	atomic.AddInt32(&s.queueLen, 1)
-	readCh := make(chan *requestf.ResponsePacket)
-	adp.resp.Store(msg.Req.IRequestId, readCh)
+	msg.RespCh = make(chan *requestf.ResponsePacket)
+	adp.resp.Store(msg.Req.IRequestId, msg.RespCh)
 	var releaseFunc = func() {
 		CheckPanic()
 		atomic.AddInt32(&s.queueLen, -1)
@@ -265,7 +263,7 @@ func (s *ServantProxy) doInvoke(ctx context.Context, msg *Message, timeout time.
 	if msg.Async {
 		go func() {
 			defer releaseFunc()
-			err := s.waitResp(msg, adp, timeout, needCheck)
+			err := s.waitResp(msg, timeout, needCheck)
 			s.manager.postInvoke()
 			msg.End()
 			s.reportStat(msg, err)
@@ -280,20 +278,18 @@ func (s *ServantProxy) doInvoke(ctx context.Context, msg *Message, timeout time.
 		return nil
 	}
 
-	return s.waitResp(msg, adp, timeout, needCheck)
+	return s.waitResp(msg, timeout, needCheck)
 }
 
-func (s *ServantProxy) waitResp(msg *Message, adp *AdapterProxy, timeout time.Duration, needCheck bool) error {
-	ch, _ := adp.resp.Load(msg.Req.IRequestId)
-	readCh := ch.(chan *requestf.ResponsePacket)
-
+func (s *ServantProxy) waitResp(msg *Message, timeout time.Duration, needCheck bool) error {
+	adp := msg.Adp
 	select {
 	case <-rtimer.After(timeout):
 		msg.Status = basef.TARSINVOKETIMEOUT
 		adp.failAdd()
 		return fmt.Errorf("request timeout, begin time:%d, cost:%d, obj:%s, func:%s, addr:(%s:%d), reqid:%d",
 			msg.BeginTime, msg.Cost(), msg.Req.SServantName, msg.Req.SFuncName, adp.point.Host, adp.point.Port, msg.Req.IRequestId)
-	case msg.Resp = <-readCh:
+	case msg.Resp = <-msg.RespCh:
 		if needCheck {
 			go func() {
 				adp.reset()