[ 
http://jira.amdatu.org/jira/browse/AMDATU-372?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Bram de Kruijff resolved AMDATU-372.
------------------------------------

    Fix Version/s:     (was: Backlog)
       Resolution: Won't Fix

Old review. Code has been refactorred and reviewed.
                
> Concurrency issues with optional dependencies in 
> HttpContextManagerServiceImpl, ResourceProviderListener and 
> WinkRegistrationServiceImpl
> ----------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: AMDATU-372
>                 URL: http://jira.amdatu.org/jira/browse/AMDATU-372
>             Project: Amdatu
>          Issue Type: Bug
>          Components: Amdatu Web
>    Affects Versions: 0.3.0
>            Reporter: Ivo Ladage - van Doorn
>
> When a service defines optional dependencies, the methods onAdded and 
> onRemoved can be invoked concurrently with the invocation of destroy() on 
> that service. This means that the implementation of the destroy and 
> onAdded/onRemoved must be prepared to be invoked simultaneously by different 
> threads. The following classes therefore need to be refactored as they do not 
> properly handle concurrency:
> HttpContextManagerServiceImpl
> - addResourceProvider can be invoked concurrently,  so 
> getOrCreateContextComponentTuple should at least be synchronized
> ResourceProviderListener 
> - stop and resourceProviderAdded can be invoked concurrently
> WinkRegistrationServiceImpl
> - onAdded and destroy can be invoked concurrently

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
http://jira.amdatu.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        
_______________________________________________
Amdatu-developers mailing list
[email protected]
http://lists.amdatu.org/mailman/listinfo/amdatu-developers

Reply via email to