make some logs meaning full

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

Branch: refs/heads/4.0.0-grouping
Commit: 3d446201a38a41d29ac5120a1a0f9f5fe028f211
Parents: 6e7b8f9
Author: Udara Liyanage <[email protected]>
Authored: Thu Sep 25 18:10:30 2014 +0530
Committer: Udara Liyanage <[email protected]>
Committed: Thu Sep 25 18:10:30 2014 +0530

----------------------------------------------------------------------
 .../stratos/manager/manager/CartridgeSubscriptionManager.java   | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/stratos/blob/3d446201/components/org.apache.stratos.manager/src/main/java/org/apache/stratos/manager/manager/CartridgeSubscriptionManager.java
----------------------------------------------------------------------
diff --git 
a/components/org.apache.stratos.manager/src/main/java/org/apache/stratos/manager/manager/CartridgeSubscriptionManager.java
 
b/components/org.apache.stratos.manager/src/main/java/org/apache/stratos/manager/manager/CartridgeSubscriptionManager.java
index c3b27c7..cfc1504 100644
--- 
a/components/org.apache.stratos.manager/src/main/java/org/apache/stratos/manager/manager/CartridgeSubscriptionManager.java
+++ 
b/components/org.apache.stratos.manager/src/main/java/org/apache/stratos/manager/manager/CartridgeSubscriptionManager.java
@@ -102,7 +102,7 @@ public class CartridgeSubscriptionManager {
             appSubscription = 
dataInsertionAndRetrievalMgr.getApplicationSubscription(tenantId, appId);
 
         } catch (PersistenceManagerException e) {
-            log.error("failed to Application Subscription for appId: " + appId 
+ " and tenantId: " + tenantId + " e:" + e);
+            log.error("failed to retrieve application Subscription for appId: 
" + appId + " and tenantId: " + tenantId + " e:" + e);
             throw new ApplicationSubscriptionException(e);
         }
 
@@ -122,9 +122,8 @@ public class CartridgeSubscriptionManager {
                                
this.persistApplicationSubscription(appSubscription);
                        } catch (ADCException e) {
                                // TODO Auto-generated catch block
-                               log.error("app Id already exists, failed to 
createCompositeAppSubscription for appId: " + appId + 
+                               log.error("Failed to persist applicaiton 
subscription for appId: " + appId +
                                                                                
                                                                        " and 
tenantId: " + tenantId + " e:" + e);
-                               e.printStackTrace();
                        }
         }
 

Reply via email to