[jira] Updated: (CAY-1165) Stop unwinding runtime exceptions

["Andrus Adamchik (JIRA)" <ji...@apache.org>]

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

Andrus Adamchik updated CAY-1165:
---------------------------------

    Priority: Minor  (was: Major)

> Stop unwinding runtime exceptions
> ---------------------------------
>
>                 Key: CAY-1165
>                 URL: https://issues.apache.org/jira/browse/CAY-1165
>             Project: Cayenne
>          Issue Type: Improvement
>          Components: Cayenne Core Library
>    Affects Versions: 3.0M5
>            Reporter: Andrus Adamchik
>            Assignee: Andrus Adamchik
>            Priority: Minor
>
> The practice of unwinding Cayenne exceptions seemed like  a good idea at the time. This would look like:
>        try {
>           ....
>         }
>         catch (Exception ex) {
>             throw new CayenneException("Exception", Util.unwindException(ex));
>         }
> And was supposed to trim Exception chain to just the underlying cause plus a Cayenne[Runtime]Exception wrapper. However in many cases intermediate places where Cayenne catches exception, wraps it and rethrows, provide additional forensics info helping to identify the cause. For instance FieldAccessor shows which field has failed:
>        try {
>             field.set(object, newValue);
>         }
>         catch (Throwable th) {
>             throw new PropertyException(
>                     "Error writing field: " + field.getName(),
>                     this,
>                     object,
>                     th);
>         }
> So I think we should get rid of all calls to Util.unwindException(..) and rethrow it unchanged.

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