diff options
author | Pamela Dragosh <pdragosh@research.att.com> | 2017-09-12 10:28:35 -0400 |
---|---|---|
committer | Pamela Dragosh <pdragosh@research.att.com> | 2017-09-12 12:46:46 -0400 |
commit | 65dff31a2e34b14347f1c0311420463c367afd3d (patch) | |
tree | bbd5dba3e1f48449c43085ef373e6ca235760bbc /integrity-monitor/src/main/java/org/onap/policy/common/im/StateTransition.java | |
parent | 5827bc8c28d85f536ac2dff3c4e03ab83f0f8408 (diff) |
Fix sonar critical associated with logging
Addressing the following sonar critical:
Use a logger to log this exception
Either log or rethrow this exception
It does not hurt to simply do these as well as clean out
lines of code that dump to Sys.out via e.printStackTrace.
Issue-ID: POLICY-195
Change-Id: Ibc328094a0653b7baa04778298e4a69955105ea3
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>
Diffstat (limited to 'integrity-monitor/src/main/java/org/onap/policy/common/im/StateTransition.java')
-rw-r--r-- | integrity-monitor/src/main/java/org/onap/policy/common/im/StateTransition.java | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/integrity-monitor/src/main/java/org/onap/policy/common/im/StateTransition.java b/integrity-monitor/src/main/java/org/onap/policy/common/im/StateTransition.java index a45a536b..03f6f4e3 100644 --- a/integrity-monitor/src/main/java/org/onap/policy/common/im/StateTransition.java +++ b/integrity-monitor/src/main/java/org/onap/policy/common/im/StateTransition.java @@ -55,6 +55,7 @@ public class StateTransition { setupStateTable(); } catch(Exception ex) { + logger.debug(ex); throw new StateTransitionException("StateTransition Exception: " + ex.toString()); } } @@ -142,7 +143,7 @@ public class StateTransition { stateElement.displayStateElement(); } catch(Exception ex) { - logger.error("String split exception: " + ex.toString()); + logger.error("String split exception: " + ex); } } else { @@ -152,6 +153,7 @@ public class StateTransition { throw new StateTransitionException(msg); } } catch (Exception ex) { + logger.debug(ex); throw new StateTransitionException("Exception: " + ex.toString() + ", adminState=[" + adminState + "], opState=[" + opState + "], availStatus=[" + availStatus + "], standbyStatus=[" + standbyStatus + "], actionName=[" + actionName + "]"); } |