[
https://issues.apache.org/jira/browse/CLOUDSTACK-10334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16417122#comment-16417122
]
ASF subversion and git services commented on CLOUDSTACK-10334:
--------------------------------------------------------------
Commit c1c587fffe0dc3cc703829a4a4501afb1fee1a45 in cloudstack's branch
refs/heads/master from lzh3636
[ https://gitbox.apache.org/repos/asf?p=cloudstack.git;h=c1c587f ]
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
> Inadequate information for handling catch clauses
> -------------------------------------------------
>
> Key: CLOUDSTACK-10334
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10334
> Project: CloudStack
> Issue Type: Improvement
> Security Level: Public(Anyone can view this level - this is the
> default.)
> Reporter: Zhenhao Li
> Priority: Major
> Labels: easyfix
>
> Their are some situations that different exception types are caught, but the
> handling of those exceptions can not show the differences of those types.
> Here are the code snippets we found which have this problem:
> *cloudstack/server/src/main/java/com/cloud/api/dispatch/ParamProcessWorker.java*
> [https://github.com/apache/cloudstack/blob/893a88d225276e45f12f9490e6af2c94a81c2965/server/src/main/java/com/cloud/api/dispatch/ParamProcessWorker.java]
> At Line *261* and Line *265.* We can see that two exception types are caught,
> but the logging statements here can not show the exception type at all.
> Also they threw new exceptions after the logs, but the throw statements in
> these two catch clauses are identical, which are not distinguishable.
> It may cause confusions to the person who is reading the log, the person can
> not know what exception happened here and can not distinguish logs generated
> by these two statements.
> Maybe adding stack trace information to these two logging statements and
> change the log message to handle specific situations is a simple way to
> improve it.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
