[jira] Updated: (DERBY-2581) Callers of SanityManager.THROWASSERT should chain the exceptions when possible

["Knut Anders Hatlen (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/DERBY-2581?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Knut Anders Hatlen updated DERBY-2581:
--------------------------------------

    Attachment: chain.stat
                chain.diff

The attached patch changes all the THROWASSERTs I found inside catch blocks so that the original exceptions are chained and preserved. Derbyall and suites.All passed.

> Callers of SanityManager.THROWASSERT should chain the exceptions when possible
> ------------------------------------------------------------------------------
>
>                 Key: DERBY-2581
>                 URL: https://issues.apache.org/jira/browse/DERBY-2581
>             Project: Derby
>          Issue Type: Improvement
>    Affects Versions: 10.3.0.0
>            Reporter: Knut Anders Hatlen
>         Assigned To: Knut Anders Hatlen
>            Priority: Minor
>         Attachments: chain.diff, chain.stat
>
>
> Some places in the code where SanityManager.THROWASSERT is used to wrap unexpected exceptions, the underlying exception is not linked to the assert exception. Therefore, the original stack trace, and possibly the message text, are lost. THROWASSERT(Throwable) or THROWASSERT(String,THROWABLE) should be used instead in these cases.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.