[jira] [Commented] (IGNITE-13290) SpringTransactionManager adds support for nested transactions

["Ivan Pavlukhin (Jira)" <ji...@apache.org>]

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

Ivan Pavlukhin commented on IGNITE-13290:
-----------------------------------------

Hi [~liyuj], issue summary confuses with [NESTED|https://docs.spring.io/spring-framework/docs/current/javadoc-api/org/springframework/transaction/annotation/Propagation.html#NESTED] transaction propagation. But as I see from description the issue is mostly about REQUIRES_NEW support. I suggest to rename it to "SpringTransactionManager adds support for REQUIRES_NEW transaction propagation". What do you think?

> SpringTransactionManager adds support for nested transactions
> -------------------------------------------------------------
>
>                 Key: IGNITE-13290
>                 URL: https://issues.apache.org/jira/browse/IGNITE-13290
>             Project: Ignite
>          Issue Type: Bug
>          Components: spring
>    Affects Versions: 2.8.1
>            Reporter: YuJue Li
>            Priority: Minor
>             Fix For: 2.10
>
>
> If the transaction propagation of the outer method is REQUIRED, the transaction propagation of the inner method is REQUIRES_NEW, the following error will be prompted:
> org.springframework.transaction.TransactionSuspensionNotSupportedException: Transaction manager [org.apache.ignite.transactions.spring.SpringTransactionManager] does not support transaction suspension
> But we see, Ignite's org.apache.ignite.transactions.Transaction, there are suspend() and resume() methods, and the function is normal, so SpringTransactionManager should support similar functions.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)