From: Timoney, Dan (dt5972) <dtimoney@att.com>
Date: Fri, 9 Nov 2018 20:48:47 +0000 (-0500)
Subject: Fix node id / type in log entry
X-Git-Tag: 1.0.4~9^2~184
X-Git-Url: https://gerrit.onap.org/r/gitweb?a=commitdiff_plain;h=5f3cb802bec2fdbfa6e78518605257bff72615cb;p=ccsdk%2Fsli.git

Fix node id / type in log entry

Node id and type in log entries are not correct for nodes within a block,
for example.

Change-Id: I4023de08215ba6de5bfdd2a561048d9fe84201e5
Issue-ID: CCSDK-657
Signed-off-by: Timoney, Dan (dt5972) <dtimoney@att.com>
---

diff --git a/sli/provider/src/main/java/org/onap/ccsdk/sli/core/sli/provider/SvcLogicServiceImpl.java b/sli/provider/src/main/java/org/onap/ccsdk/sli/core/sli/provider/SvcLogicServiceImpl.java
index dd7139a7b..c5a848a17 100755
--- a/sli/provider/src/main/java/org/onap/ccsdk/sli/core/sli/provider/SvcLogicServiceImpl.java
+++ b/sli/provider/src/main/java/org/onap/ccsdk/sli/core/sli/provider/SvcLogicServiceImpl.java
@@ -154,8 +154,6 @@ public class SvcLogicServiceImpl implements SvcLogicService {
         LOG.info("About to execute graph {}", graph.toString());
 		try {
 			while (curNode != null) {
-				MDC.put("nodeId", curNode.getNodeId() + " (" + curNode.getNodeType() + ")");
-				LOG.info("About to execute node # {} ({})", curNode.getNodeId(), curNode.getNodeType());
 
 				SvcLogicNode nextNode = executeNode(curNode, ctx);
 				curNode = nextNode;
@@ -174,6 +172,8 @@ public class SvcLogicServiceImpl implements SvcLogicService {
             return (null);
         }
 
+		LOG.info("About to execute node # {} ({})", node.getNodeId(), node.getNodeType());
+		
         if (LOG.isDebugEnabled()) {
             LOG.debug("Executing node {}", node.getNodeId());
         }
@@ -183,6 +183,8 @@ public class SvcLogicServiceImpl implements SvcLogicService {
         if (executor != null) {
             LOG.debug("Executing node executor for node type {} - {}", node.getNodeType(),
                     executor.getClass().getName());
+
+    		MDC.put("nodeId", node.getNodeId() + " (" + node.getNodeType() + ")");
             return (executor.execute(this, node, ctx));
         } else {
             throw new SvcLogicException("Attempted to execute a node of type " + node.getNodeType() + ", but no executor was registered for this type");