[jboss-jira] [JBoss JIRA] Commented: (JBPORTAL-2098) Last-login date feature creates deadlocks

Wed Aug 20 07:07:23 EDT 2008

    [ https://jira.jboss.org/jira/browse/JBPORTAL-2098?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12425812#action_12425812 ] 

Thomas Heute commented on JBPORTAL-2098:
----------------------------------------

Changed on 2.7 Branch.
Now it stores the *login* date, also it has changed from a display string to the 'time' (long)

> Last-login date feature creates deadlocks
> -----------------------------------------
>
>                 Key: JBPORTAL-2098
>                 URL: https://jira.jboss.org/jira/browse/JBPORTAL-2098
>             Project: JBoss Portal
>          Issue Type: Bug
>      Security Level: Public(Everyone can see) 
>          Components: Portal Core
>    Affects Versions: 2.6.5 SP1, 2.6.6 Final, 2.7.0 Alpha1, 2.7.0 Beta1
>            Reporter: Thomas Heute
>            Assignee: Sohil Shah
>             Fix For: 2.7.0 CR1 , 2.6.7 Final
>
>
> The code in the User Interceptor that set the last time a user was logged is creating locks.
> In fact threads are trying to acquire a write lock on the user.
> I disabled it for a customer, now we need a generic solution.
> I think to remember that the reason it is done that way is that someone wanted the date when the session stops instead of when it starts.
> Sohil, could you have a look and either:
>  - keep that feature but done in a different way (i don't see how)
>  - record the last login date (instead of last logout) i think this is acceptable in many scenario, anyway the other solution is not perfect neither, most people never log out so it records the session timeout date.
> - move the code to another interceptor so that it can be easily turned off and turn it off by default in 2.7

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira