[ 
http://jira.magnolia.info/browse/MAGNOLIA-2427?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18599#action_18599
 ] 

Jan Haderka commented on MAGNOLIA-2427:
---------------------------------------

Hi Philippe,
the patch looks great, but at this point we would like to not introduce yet 
another library for handling concurrency (we already use edu.oswego.* at 
various places). We will revisit this and apply (And modify to use 
java.util.concurrent) once we are dropping 1.4 support and moving to 1.5.

> replace custom thread pool with library
> ---------------------------------------
>
>                 Key: MAGNOLIA-2427
>                 URL: http://jira.magnolia.info/browse/MAGNOLIA-2427
>             Project: Magnolia
>          Issue Type: Improvement
>          Components: activation
>    Affects Versions: 3.6.3
>            Reporter: Philippe Marschall
>            Assignee: Philipp Bracher
>            Priority: Minor
>         Attachments: ThreadPool.java.patch
>
>
> The module exchange-simple uses it's own thread pool instead of a library. 
> There is no need for this since there are mature, stable, tested and debugged 
> libraries available that do this. The attached patch uses the backport of 
> java.util.concurrent that as an easy migration path to JDK 1.5 in Magnlia 4.
> The patch also removes two unused variables and corrects some indentation 
> issues.

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

        

----------------------------------------------------------------
for list details see
http://documentation.magnolia.info/
----------------------------------------------------------------

Reply via email to