[jira] [Comment Edited] (IGNITE-4767) rollback exception hides the origin exception (e.g. commit)

["Dmitriy Pavlov (JIRA)" <ji...@apache.org>]

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

Dmitriy Pavlov edited comment on IGNITE-4767 at 7/26/17 5:07 PM:
-----------------------------------------------------------------

I've restarted tests for 2.0 group using RunAll chain: 
http://ci.ignite.apache.org/project.html?projectId=Ignite20Tests&branch_Ignite20Tests=pull%2F1599%2Fhead


was (Author: dpavlov):
I've restarted tests for 2.0: 
http://ci.ignite.apache.org/project.html?projectId=Ignite20Tests&branch_Ignite20Tests=pull%2F1599%2Fhead

> 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: Dmitriy Pavlov
>             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)