[ 
https://issues.apache.org/jira/browse/IGNITE-4767?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16103111#comment-16103111
 ] 

Dmitriy Pavlov edited comment on IGNITE-4767 at 8/9/17 2:36 PM:
----------------------------------------------------------------

Merged to master


was (Author: dpavlov):
Merged to master

Is to be resolved after PR to merged to master

> rollback exception hides the origin exception (e.g. commit)
> -----------------------------------------------------------
>
>                 Key: IGNITE-4767
>                 URL: https://issues.apache.org/jira/browse/IGNITE-4767
>             Project: Ignite
>          Issue Type: Bug
>          Components: cache, general
>    Affects Versions: 1.8
>            Reporter: Alexandr Kuramshin
>            Assignee: Alexandr Kuramshin
>             Fix For: 2.2
>
>
> There is too much code places like:
> {noformat}
> try {
>       return txFuture.get();
> }
> catch (IgniteCheckedException e) {
>       tx.rollbackAsync();
>       throw e;
> }
> {noformat}
> where an error upon rollback hides the actual exception {{e}}.
> This should be implemented in the way like try-with-resources does:
> {noformat}
> try {
>       return txFuture.get();
> }
> catch (IgniteCheckedException e1) {
>       try {
>               tx.rollbackAsync();
>       }
>       catch (Throwable inner) {
>               e.addSuppressed(inner);
>       }
>       throw e;
> }
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to