Fix for xApp transction release place 22/7022/1
authorAnssi Mannila <anssi.mannila@nokia.com>
Wed, 6 Oct 2021 11:21:48 +0000 (14:21 +0300)
committerAnssi Mannila <anssi.mannila@nokia.com>
Tue, 9 Nov 2021 08:05:23 +0000 (10:05 +0200)
 - xApp trasaction release is now done before REST notification is sent to xApp
 - This makes subscription deletion possible instantly after notification

Change-Id: Ia076180aaff40a0b8f8f3c655895a5e37364c5bd
Signed-off-by: Anssi Mannila <anssi.mannila@nokia.com>
pkg/control/control.go

index f67f4e2..5dedb1e 100755 (executable)
@@ -452,7 +452,8 @@ func (c *Control) processSubscriptionRequests(restSubscription *RESTSubscription
 
                subRespMsg, err := c.handleSubscriptionRequest(trans, &subReqMsg, meid, *restSubId)
 
-               xapp.Logger.Info("Handled SubscriptionRequest index=%v, %s", index, idstring(nil, trans))
+               xapp.Logger.Debug("Handled SubscriptionRequest index=%v, %s", index, idstring(nil, trans))
+               trans.Release()
 
                if err != nil {
                        c.sendUnsuccesfullResponseNotification(restSubId, restSubscription, xAppEventInstanceID, err, clientEndpoint, trans)
@@ -463,7 +464,6 @@ func (c *Control) processSubscriptionRequests(restSubscription *RESTSubscription
                                index, *restSubId, clientEndpoint.Host, *clientEndpoint.HTTPPort, xAppEventInstanceID, e2EventInstanceID, idstring(nil, trans))
                        c.sendSuccesfullResponseNotification(restSubId, restSubscription, xAppEventInstanceID, e2EventInstanceID, clientEndpoint, trans)
                }
-               trans.Release()
        }
 }