Add exception to logs

Fixing the comments from previous review - https://gerrit.onap.org/r/c/policy/apex-pdp/+/116833

Issue-ID: POLICY-2885
Change-Id: I85474b365d92771e6e1b56bba95bde6e319d4ec2
Signed-off-by: Ram Krishna Verma <ram_krishna.verma@bell.ca>

diff --git a/plugins/plugins-event/plugins-event-protocol/plugins-event-protocol-jms/src/main/java/org/onap/policy/apex/plugins/event/protocol/jms/Apex2JmsObjectEventConverter.java b/plugins/plugins-event/plugins-event-protocol/plugins-event-protocol-jms/src/main/java/org/onap/policy/apex/plugins/event/protocol/jms/Apex2JmsObjectEventConverter.java
index 0f3a21b..89e2e81 100644 (file)
--- a/plugins/plugins-event/plugins-event-protocol/plugins-event-protocol-jms/src/main/java/org/onap/policy/apex/plugins/event/protocol/jms/Apex2JmsObjectEventConverter.java
+++ b/plugins/plugins-event/plugins-event-protocol/plugins-event-protocol-jms/src/main/java/org/onap/policy/apex/plugins/event/protocol/jms/Apex2JmsObjectEventConverter.java
@@ -80,19 +80,19 @@ public final class Apex2JmsObjectEventConverter implements ApexEventProtocolConv
         Method getObjectMethod;
         try {
             getObjectMethod = eventObject.getClass().getMethod("getObject", (Class<?>[]) null);
-        } catch (Exception e) {
+        } catch (Exception exp) {
             final String errorMessage = "message \"" + eventObject
                             + "\" received from JMS does not have a \"getObject()\" method";
-            throw new ApexEventRuntimeException(errorMessage);
+            throw new ApexEventRuntimeException(errorMessage, exp);
         }
 
         Object jmsIncomingObject;
         try {
             jmsIncomingObject = getObjectMethod.invoke(eventObject, (Object[]) null);
-        } catch (final Exception e) {
+        } catch (final Exception exp) {
             final String errorMessage = "object contained in message \"" + eventObject
                             + "\" received from JMS could not be retrieved as a Java object";
-            throw new ApexEventRuntimeException(errorMessage, e);
+            throw new ApexEventRuntimeException(errorMessage, exp);
         }
 
         // Check that the consumer parameters for JMS->Apex messaging have been set

diff --git a/plugins/plugins-event/plugins-event-protocol/plugins-event-protocol-jms/src/main/java/org/onap/policy/apex/plugins/event/protocol/jms/Apex2JmsTextEventConverter.java b/plugins/plugins-event/plugins-event-protocol/plugins-event-protocol-jms/src/main/java/org/onap/policy/apex/plugins/event/protocol/jms/Apex2JmsTextEventConverter.java
index e9f9109..701d61e 100644 (file)
--- a/plugins/plugins-event/plugins-event-protocol/plugins-event-protocol-jms/src/main/java/org/onap/policy/apex/plugins/event/protocol/jms/Apex2JmsTextEventConverter.java
+++ b/plugins/plugins-event/plugins-event-protocol/plugins-event-protocol-jms/src/main/java/org/onap/policy/apex/plugins/event/protocol/jms/Apex2JmsTextEventConverter.java
@@ -47,20 +47,20 @@ public final class Apex2JmsTextEventConverter extends Apex2JsonEventConverter {
         Method getTextMethod;
         try {
             getTextMethod = eventObject.getClass().getMethod("getText", (Class<?>[]) null);
-        } catch (Exception e) {
+        } catch (Exception exp) {
             final String errorMessage = "message \"" + eventObject
                             + "\" received from JMS does not have a \"getText()\" method";
-            throw new ApexEventRuntimeException(errorMessage);
+            throw new ApexEventRuntimeException(errorMessage, exp);
         }
 
 
         String jmsString;
         try {
             jmsString = (String) getTextMethod.invoke(eventObject, (Object[]) null);
-        } catch (final Exception e) {
+        } catch (final Exception exp) {
             final String errorMessage = "object contained in message \"" + eventObject
                     + "\" received from JMS could not be retrieved as a Java String";
-            throw new ApexEventRuntimeException(errorMessage, e);
+            throw new ApexEventRuntimeException(errorMessage, exp);
         }
 
         // Use the generic JSON plugin from here