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

ASF subversion and git services commented on ISIS-1414:
-------------------------------------------------------

Commit 21b27fe49790adc78a3fe7804ff4feba7790bbed in isis's branch 
refs/heads/ISIS-1414 from [~danhaywood]
[ https://git-wip-us.apache.org/repos/asf?p=isis.git;h=21b27fe ]

ISIS-1414: moving the @RequestScoped initialization/close stuff out of 
IsisTransactionManager and IsisTransaction, and into PersistenceSession.

Also:
- CommandService#startTransaction is no longer called.
- remove unused code/hashmap in PersistenceSession


> nextTransaction can cause null pointers with the new MetricsService. 
> ---------------------------------------------------------------------
>
>                 Key: ISIS-1414
>                 URL: https://issues.apache.org/jira/browse/ISIS-1414
>             Project: Isis
>          Issue Type: Bug
>          Components: Core
>    Affects Versions: 1.12.1
>            Reporter: Dan Haywood
>            Assignee: Dan Haywood
>            Priority: Minor
>             Fix For: 1.13.0
>
>
> The reason is that request-scoped services are implemented as transaction 
> services.  Most of the time the scope of a request and the scope of a 
> transacion are one and the same: there is just one transaction per scope.  
> However, TransactionService#nextTransaction admits for there being multiple 
> transactions per request.
> This bug is that the current implementation for RequestScoped does the 
> init/close as part of the Transaction management.  Instead, this should be 
> part of the PersistenceSession#open/close (a PersistenceSession wraps a JDO 
> session, so basically corresponds to the request).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to