You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by ra...@apache.org on 2018/03/28 10:02:22 UTC

[cloudstack] branch master updated: CLOUDSTACK-10334: Fix inadequate information for handling catch clauses (#2510)

This is an automated email from the ASF dual-hosted git repository.

rafael pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack.git


The following commit(s) were added to refs/heads/master by this push:
     new c1c587f  CLOUDSTACK-10334: Fix inadequate information for handling catch clauses (#2510)
c1c587f is described below

commit c1c587fffe0dc3cc703829a4a4501afb1fee1a45
Author: lzh3636 <li...@outlook.com>
AuthorDate: Wed Mar 28 06:02:16 2018 -0400

    CLOUDSTACK-10334: Fix inadequate information for handling catch clauses (#2510)
    
    * Add log stack traces
    
    * Change log stack traces
    
    Add original stack trace to the new throw and delete the error logs
---
 server/src/main/java/com/cloud/api/dispatch/ParamProcessWorker.java | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/server/src/main/java/com/cloud/api/dispatch/ParamProcessWorker.java b/server/src/main/java/com/cloud/api/dispatch/ParamProcessWorker.java
index feefaab..f030209 100644
--- a/server/src/main/java/com/cloud/api/dispatch/ParamProcessWorker.java
+++ b/server/src/main/java/com/cloud/api/dispatch/ParamProcessWorker.java
@@ -258,13 +258,11 @@ public class ParamProcessWorker implements DispatchWorker {
                 }
 
             } catch (final IllegalArgumentException e) {
-                s_logger.error("Error initializing command " + cmd.getCommandName() + ", field " + field.getName() + " is not accessible.");
                 throw new CloudRuntimeException("Internal error initializing parameters for command " + cmd.getCommandName() + " [field " + field.getName() +
-                        " is not accessible]");
+                        " is not accessible]", e);
             } catch (final IllegalAccessException e) {
-                s_logger.error("Error initializing command " + cmd.getCommandName() + ", field " + field.getName() + " is not accessible.");
                 throw new CloudRuntimeException("Internal error initializing parameters for command " + cmd.getCommandName() + " [field " + field.getName() +
-                        " is not accessible]");
+                        " is not accessible]", e);
             }
 
         }

-- 
To stop receiving notification emails like this one, please contact
rafael@apache.org.