Repository: ambari
Updated Branches:
  refs/heads/branch-1.7.0 b8a1f3a6e -> 3e7ba6190


AMBARI-8030 - Views: framework should log view internal exceptions (tbeerbower)


Project: http://git-wip-us.apache.org/repos/asf/ambari/repo
Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/3e7ba619
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/3e7ba619
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/3e7ba619

Branch: refs/heads/branch-1.7.0
Commit: 3e7ba619088a50fa83c9e8c5eef713a9e5e6f1ac
Parents: b8a1f3a
Author: tbeerbower <[email protected]>
Authored: Thu Oct 30 04:59:22 2014 -0400
Committer: tbeerbower <[email protected]>
Committed: Thu Oct 30 04:59:54 2014 -0400

----------------------------------------------------------------------
 .../org/apache/ambari/server/view/ViewSubResourceProvider.java   | 4 ++++
 1 file changed, 4 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/3e7ba619/ambari-server/src/main/java/org/apache/ambari/server/view/ViewSubResourceProvider.java
----------------------------------------------------------------------
diff --git 
a/ambari-server/src/main/java/org/apache/ambari/server/view/ViewSubResourceProvider.java
 
b/ambari-server/src/main/java/org/apache/ambari/server/view/ViewSubResourceProvider.java
index 260fc3d..8e1ccbc 100644
--- 
a/ambari-server/src/main/java/org/apache/ambari/server/view/ViewSubResourceProvider.java
+++ 
b/ambari-server/src/main/java/org/apache/ambari/server/view/ViewSubResourceProvider.java
@@ -112,6 +112,7 @@ public class ViewSubResourceProvider extends 
AbstractResourceProvider {
       } catch (org.apache.ambari.view.ResourceAlreadyExistsException e) {
         throw new ResourceAlreadyExistsException(e.getMessage());
       } catch (Exception e) {
+        LOG.error("Caught exception creating view sub resources.", e);
         throw new SystemException(e.getMessage(), e);
       }
     }
@@ -177,6 +178,7 @@ public class ViewSubResourceProvider extends 
AbstractResourceProvider {
     } catch (org.apache.ambari.view.UnsupportedPropertyException e) {
       throw new UnsupportedPropertyException(getResourceType(e), 
e.getPropertyIds());
     } catch (Exception e) {
+      LOG.error("Caught exception getting view sub resources.", e);
       throw new SystemException(e.getMessage(), e);
     }
   }
@@ -201,6 +203,7 @@ public class ViewSubResourceProvider extends 
AbstractResourceProvider {
         } catch (org.apache.ambari.view.UnsupportedPropertyException e) {
           throw new UnsupportedPropertyException(getResourceType(e), 
e.getPropertyIds());
         } catch (Exception e) {
+          LOG.error("Caught exception updating view sub resources.", e);
           throw new SystemException(e.getMessage(), e);
         }
       }
@@ -224,6 +227,7 @@ public class ViewSubResourceProvider extends 
AbstractResourceProvider {
       } catch (org.apache.ambari.view.UnsupportedPropertyException e) {
         throw new UnsupportedPropertyException(getResourceType(e), 
e.getPropertyIds());
       } catch (Exception e) {
+        LOG.error("Caught exception deleting view sub resources.", e);
         throw new SystemException(e.getMessage(), e);
       }
     }

Reply via email to