Logging exceptions in ArtifactHandlerProvider 53/29053/6
authorJakub Dudycz <jakub.dudycz@nokia.com>
Wed, 24 Jan 2018 17:29:35 +0000 (18:29 +0100)
committerPatrick Brady <pb071s@att.com>
Thu, 25 Jan 2018 19:06:13 +0000 (19:06 +0000)
Update

Change-Id: I4214b99b84f951a6b508ff40d599585a092467f5
Issue-ID: APPC-495
Signed-off-by: Jakub Dudycz <jakub.dudycz@nokia.com>
appc-inbound/appc-artifact-handler/provider/src/main/java/org/onap/appc/artifact/handler/ArtifactHandlerProvider.java

index 51f286e..dc6489d 100644 (file)
@@ -92,7 +92,7 @@ public class ArtifactHandlerProvider implements AutoCloseable, ArtifactHandlerSe
         try {
             ArtifactHandlerProviderUtil.loadProperties();
         } catch (Exception e) {
-            log.error("Caught Exception while trying to load properties file");
+            log.error("Caught exception while trying to load properties file", e);
         }
         // Listener for changes to Services tree
 
@@ -110,11 +110,10 @@ public class ArtifactHandlerProvider implements AutoCloseable, ArtifactHandlerSe
         try {
             CheckedFuture<Void, TransactionCommitFailedException> checkedFuture = t.submit();
             checkedFuture.get();
-            log.info("Create Containers succeeded!: ");
+            log.info("Create containers succeeded!");
 
         } catch (InterruptedException | ExecutionException e) {
-            log.error("Create Containers Failed: " + e);
-            e.printStackTrace();
+            log.error("Create containers failed",  e);
         }
     }