[gatein-issues] [JBoss JIRA] (GTNPC-82) Potential concurency issues in FederatingPortletInvokerService and ContainerPortletInvoker

[Nicolas Filotto (JIRA)]

Nicolas Filotto created GTNPC-82:
------------------------------------

             Summary: Potential concurency issues in FederatingPortletInvokerService and ContainerPortletInvoker
                 Key: GTNPC-82
                 URL: https://issues.jboss.org/browse/GTNPC-82
             Project: GateIn Portlet Container
          Issue Type: Quality Risk
    Affects Versions: 2.3.0-GA
            Reporter: Nicolas Filotto


While debugging to undestand why the PC could not find my portlets, I found the next potential concurrency issues :
# In FederatingPortletInvokerService.getOrResolveFederatedInvoker() the invokerCache is directly modified instead of being recreated from scratch an reassigned which could cause HashMap corruptions under heavy load 
# In ContainerPortletInvoker: The methods addPortletContainer and removePortletContainer should be synchronized to prevent any modification loss in case of concurrent modifications. Moreover the variable portlets should be volatile for the same reason

This must be seen as a feedback you are free to resolve it as won't fix if you don't find it relevant.

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