[
https://issues.apache.org/jira/browse/PIVOT-35?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Dirk Moebius updated PIVOT-35:
------------------------------
Attachment: dispatcher.patch
This patch changes Dispatcher to use java.util.ExecutorService. This because of
two reasons:
1. ExecutorService implementations provide all of that functionality that is
missing and marked with TODO in the current Dispatcher implementation. In fact,
my patch removes all TODOs in Dispatcher, because all of them have been dealt
with.
2. Implementing a correctly working thread pool is a hard task. No offense, but
I trust the Java core developers more than you when it comes to multi thread
programming. ;-) Not because I think they're better programmers (probably not),
but because they have a LOT of tests running up and down all the time, and
there are more eyes watching. Millions.
The patch reduces Dispatcher to a thin facade over ExecutorService. Note that
since I added a shutdown() method (you get this one for free), there's now a
way to let ApplicationContext control the dispatcher lifecycle.
For motivation of this patch see discussion here:
http://mail-archives.apache.org/mod_mbox/pivot-user/201004.mbox/%[email protected]%3e
> Complete Dispatcher support
> ---------------------------
>
> Key: PIVOT-35
> URL: https://issues.apache.org/jira/browse/PIVOT-35
> Project: Pivot
> Issue Type: Improvement
> Components: core-util
> Reporter: Greg Brown
> Fix For: 1.6
>
> Attachments: dispatcher.patch
>
>
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
