Repository: ambari
Updated Branches:
  refs/heads/trunk 44b304b45 -> 6e4c89bb0


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/6e4c89bb
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/6e4c89bb
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/6e4c89bb

Branch: refs/heads/trunk
Commit: 6e4c89bb069fc6c0723c6ce11fcb2e48fe08f3e0
Parents: 44b304b
Author: tbeerbower <[email protected]>
Authored: Thu Oct 30 04:59:22 2014 -0400
Committer: tbeerbower <[email protected]>
Committed: Thu Oct 30 05:01:19 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/6e4c89bb/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