[ 
https://issues.apache.org/jira/browse/LABS-144?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12630882#action_12630882
 ] 

Ryan McKinley commented on LABS-144:
------------------------------------

If the Map is ever manipulated after starup, it will need to be synchronized.  
For many (most?) of the normal Factory implementaions, this is not an issue -- 
all the Protocol / parser / etc, are typically known at startup.  However if we 
want to dynamically add protocols, parsers, etc at runtime, it needs to be a 
synchronized map.

Rather then synchronizing getMap(), we should make sure the map implementation 
is thread safe (when necessary).  ConcurrentHashMap or 
Collections.synchronizedMap is a better option the synchronizing the parent 
function.

All this is independent of the non-thread safe issues introduced by adding a 
lazy loading initalizer.





> Consider making dependency on Spring / Cocoon Configurator optional
> -------------------------------------------------------------------
>
>                 Key: LABS-144
>                 URL: https://issues.apache.org/jira/browse/LABS-144
>             Project: Labs
>          Issue Type: Improvement
>          Components: Droids
>            Reporter: Oleg Kalnichevski
>            Priority: Minor
>         Attachments: droids.patch
>
>
> Please consider making dependency on Spring / Cocoon Configurator optional. 
> Being a happy Spring user myself, I, nonetheless, think Droids API / Core 
> classes should not depend on a particular DI framework / configuration 
> format. It should be possible to split Droids code into API/Core module and 
> runtime with only the latter being dependent on Spring / Cocoon Configurator.
> Oleg

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to