[
https://issues.apache.org/jira/browse/GEODE-5382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16539347#comment-16539347
]
Eric Shu commented on GEODE-5382:
---------------------------------
The Geode TransactionManagerImpl was deprecated and will be removed (see
GEODE-2301). So we will put off the effort to fix this issue.
Users were advised to use third party JTAManager instead.
> Geode TransactionManagerImpl.commit() does not throw Exception when
> notifyAfterCompletion failed
> ------------------------------------------------------------------------------------------------
>
> Key: GEODE-5382
> URL: https://issues.apache.org/jira/browse/GEODE-5382
> Project: Geode
> Issue Type: Bug
> Components: transactions
> Reporter: Eric Shu
> Assignee: Eric Shu
> Priority: Major
>
> Currently, commit() method only logs the exception.
> {code:java}
> try {
> ((TransactionImpl) transactionImpl).notifyAfterCompletion(status =
> gtx.getStatus());
> } catch (Exception ge) {
> LogWriterI18n writer = TransactionUtils.getLogWriterI18n();
> if (writer.infoEnabled())
> writer.info(
>
> LocalizedStrings.TransactionManagerImpl_EXCEPTION_IN_NOTIFY_AFTER_COMPLETION_DUE_TO__0,
> ge.getMessage(), ge);
> }
> {code}
> It should throw exception to user application so that application can decide
> whether retry. Currently, data inconsistency could occur but applicate does
> not even notice about it.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
