Updated Branches: refs/heads/master b172eb21c -> b4464d88d
CLOUDSTACK-4281 : Usage - exception was hiding the db error. Make the usage job fail incase of any DB insert exceptions Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/b4464d88 Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/b4464d88 Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/b4464d88 Branch: refs/heads/master Commit: b4464d88d34f2de8a57133eedf69eaa3f5f7bdec Parents: b172eb2 Author: Kishan Kavala <[email protected]> Authored: Tue Aug 13 17:24:27 2013 +0530 Committer: Kishan Kavala <[email protected]> Committed: Tue Aug 13 17:30:46 2013 +0530 ---------------------------------------------------------------------- usage/src/com/cloud/usage/parser/NetworkUsageParser.java | 7 ++----- usage/src/com/cloud/usage/parser/VmDiskUsageParser.java | 6 +----- 2 files changed, 3 insertions(+), 10 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/cloudstack/blob/b4464d88/usage/src/com/cloud/usage/parser/NetworkUsageParser.java ---------------------------------------------------------------------- diff --git a/usage/src/com/cloud/usage/parser/NetworkUsageParser.java b/usage/src/com/cloud/usage/parser/NetworkUsageParser.java index f2f24e4..4bf62d9 100644 --- a/usage/src/com/cloud/usage/parser/NetworkUsageParser.java +++ b/usage/src/com/cloud/usage/parser/NetworkUsageParser.java @@ -129,11 +129,8 @@ public static final Logger s_logger = Logger.getLogger(NetworkUsageParser.class. } } } - try { - m_usageDao.saveUsageRecords(usageRecords); - } catch (Exception ex) { - s_logger.error("Exception in usage manager", ex); - } + + m_usageDao.saveUsageRecords(usageRecords); return true; } http://git-wip-us.apache.org/repos/asf/cloudstack/blob/b4464d88/usage/src/com/cloud/usage/parser/VmDiskUsageParser.java ---------------------------------------------------------------------- diff --git a/usage/src/com/cloud/usage/parser/VmDiskUsageParser.java b/usage/src/com/cloud/usage/parser/VmDiskUsageParser.java index 84bdfd2..b0cb2ff 100644 --- a/usage/src/com/cloud/usage/parser/VmDiskUsageParser.java +++ b/usage/src/com/cloud/usage/parser/VmDiskUsageParser.java @@ -157,11 +157,7 @@ public static final Logger s_logger = Logger.getLogger(VmDiskUsageParser.class.g } } - try { - m_usageDao.saveUsageRecords(usageRecords); - } catch (Exception ex) { - s_logger.error("Exception in usage manager", ex); - } + m_usageDao.saveUsageRecords(usageRecords); return true; }
