ignite-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dmitriy Pavlov (JIRA)" <j...@apache.org>
Subject [jira] [Comment Edited] (IGNITE-4767) rollback exception hides the origin exception (e.g. commit)
Date Wed, 09 Aug 2017 14:36:00 GMT

    [ 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:35 PM:
----------------------------------------------------------------

Merged to master

Is to be resolved after PR to merged to master


was (Author: dpavlov):
Merged to 2.1.3

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)

Mime
View raw message